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.
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
Redetermine curl and git paths where necessary #12191
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, makes sense.
Review period skipped due to |
This now shows up in
Should we be exposing the path to the shims like this? Interestingly, the path to |
No, I don't think so.
That's likely because we're doing something smarter there. |
Yep. This is where the path to brew/Library/Homebrew/system_config.rb Lines 129 to 137 in 92749f9
and this is brew/Library/Homebrew/utils/curl.rb Lines 19 to 23 in 92749f9
I'll have a look tomorrow. |
See discussion at Homebrew#12191.
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?Discovered while testing an installation of Homebrew in a 10.10 VM.
We need to redetermine the curl and git paths in two places:
HOMEBREW_FORCE_BREWED_CURL/GIT
.brew update
, so the rest of the update script can use them.