-
-
Notifications
You must be signed in to change notification settings - Fork 286
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
Don't lowercase URL when those are specified in Brewfile #672
Conversation
If a Brewfile defines a specific version of a package and this package originates form an URL, the name shouldn't be lowercase. An example of such file could be: ``` brew "https://raw.githubusercontent.com/Homebrew/homebrew-core/50c928c870001577ce6b7e28edf43e05df699852/Formula/etcd.rb" ``` Previous to this case this was falling with this error: ``` curl: (22) The requested URL returned error: 404 Not Found Error: Failure while executing; `/usr/bin/curl -q --globoff --show-error --user-agent Homebrew/2.2.11\ \(Macintosh\;\ Intel\ Mac\ OS\ X\ 10.15.4\)\ curl/7.64.1 --fail --progress-bar --retry 3 --location --remote-time --continue-at 0 --output /Users/alexandru/Library/Caches/Homebrew/Formula/etcd.rb https://raw.githubusercontent.com/homebrew/homebrew-core/50c928c870001577ce6b7e28edf43e05df699852/formula/etcd.rb` exited with 22. Here's the output: curl: (22) The requested URL returned error: 404 Not Found ``` The reason of the failure was due to the name been transformed to lowercase.
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.
We specifically don't want to support URLs in Brewfile
s so would accept modifications to this PR to error on their use.
Is there a reason why URL's aren't supported? My case here is that I need a specific version of |
Run In short: they will break and when they do they can't be fixed. We provide tooling ( |
Thanks for point that out. I know this is not the best place to ask this but I've done exactly what you've said extracted the package and create a new tap. But now it tries to build the package... Since it can't find a pre-build package in:
Is this expected or did I messed something up? When installing from an URL I can get the correct version If this is not the case I still think URL installs should be allow, if |
Somewhat expected. You'll want to remove the
There is a "warning" telling you not to use them. They are likely to be deprecated and removed in future.
This is not how Homebrew works. It's a rolling-release package manager. Read https://docs.brew.sh/Versions |
Sorry, passing on this due to reasons above. |
If a Brewfile defines a specific version of a package and this
package originates form an URL, the name shouldn't be lowercase.
An example of such file could be:
Previous to this case this was falling with this error:
The reason of the failure was due to the name been transformed
to lowercase.