-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Remove old keg_only reasons. #26700
Remove old keg_only reasons. #26700
Conversation
@ilovezfs any objections or test failures in here that seem related to this PR? |
@ilovezfs to provide more elaboration: https://travis-ci.org/Homebrew/homebrew-core/jobs/367366419 indicated that some Linux porting needed done for those and I'm not convinced it's worth investing the time for stuff that's not going to be used on any versions we support. |
I think that explains why you'd want to kill off |
Also isn't checking |
Yeh, it should be.
It feels weird to have DSL values that are specific to old versions of OS X that we no longer support. Additionally, having behaviours that differ on that versions smells like something that's more likely to break in future than matching the behaviour of later versions. |
@Homebrew/core it may be worth checking the other uses of |
Deprecating these from Homebrew/brew in Homebrew/brew#4081 so remove them here first.