Thanks to visit codestin.com
Credit goes to github.com

Skip to content

build: customizable install prefix #11714

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from
Open

build: customizable install prefix #11714

wants to merge 1 commit into from

Conversation

@scarf005
Copy link

resolves #11713
supersedes #11706

@scarf005 scarf005 requested a review from a team as a code owner September 11, 2025 15:56
@scarf005 scarf005 requested review from babakks and Copilot September 11, 2025 15:56
@scarf005 scarf005 changed the title build: customizable install PREFIX build: customizable install prefix Sep 11, 2025
@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Sep 11, 2025
Copilot
Copilot AI reviewed Sep 11, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR makes the installation prefix customizable by changing the hardcoded prefix assignment to use a conditional assignment operator, allowing users to override the default installation path via environment variables or command-line arguments.

  • Changed hardcoded prefix assignment to conditional assignment in Makefile
  • Enables customization of installation directory without modifying the Makefile

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link

Thank you for your pull request! 🎉

This PR appears to fix the following issues that are not labeled with help wanted Contributions welcome :

As outlined in our Contributing Guidelines, we expect that PRs are only created for issues that have been labeled help wanted.

While we appreciate your initiative, please note that:

  • PRs for non-help wanted issues may not be reviewed immediately as they might not align with our current priorities
  • The issue might already be assigned to a team member or planned for a specific release
  • We may need to close this PR. For example, if it conflicts with ongoing work or architectural decisions

What happens next:

  • Our team will review this PR and the associated issues
  • We may add the help wanted label to the issues, if appropriate, and review this pull request
  • In some cases, we may need to close the PR. For example, if it doesn't fit our current roadmap

Thank you for your understanding and contribution to the project! 🙏

This comment was automatically generated by cliAutomation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@babakks babakks Awaiting requested review from babakks babakks is a code owner automatically assigned from cli/code-reviewers

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

external pull request originating outside of the CLI core team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

make installation prefix configurable

2 participants

@scarf005 @cliAutomation