-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Load tap migration renames from api #17599
Conversation
These would not get loaded from the API at all meaning these were not loadable by the old names when the core formula or cask tap was not installed. We assume that most users don't have those core taps tapped which means this is broken for most everyone.
Check for the following: - Tap migration rename to core tap can be loaded by short name - Tap migration rename to core tap can be loaded by long name - Tap migration renam that clashes with existing core tap short name is ignored in favor of loading the cask/formula from the core tap
668ac93
to
fb3b13e
Compare
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 thanks @apainintheneck!
Thanks for helping fix this issue. I've tried the change myself, but it seems the brew/Library/Homebrew/cask/cask_loader.rb Line 468 in fb3b13e
brew/Library/Homebrew/formulary.rb Line 835 in fb3b13e
|
@Logicer16 Thanks for catching that! I opened a PR that should address the problem. |
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?Alternative approach to #17555 that is simpler and exactly matches the current behavior. The only downside is that the type system wasn't happy and I had to use an escape hatch.
Fixes #17234