Skip to content
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

Merged
merged 21 commits into from
Apr 17, 2018
Merged

Remove old keg_only reasons. #26700

merged 21 commits into from
Apr 17, 2018

Conversation

MikeMcQuaid
Copy link
Member

Deprecating these from Homebrew/brew in Homebrew/brew#4081 so remove them here first.

@MikeMcQuaid
Copy link
Member Author

@ilovezfs any objections or test failures in here that seem related to this PR?

@MikeMcQuaid
Copy link
Member Author

@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.

@ilovezfs
Copy link
Contributor

I think that explains why you'd want to kill off keg_only :provided_until_xcode43 and :provided_until_xcode5 but why the others?

@ilovezfs
Copy link
Contributor

Also isn't checking installed? supposed to be unnecessary now?

@MikeMcQuaid
Copy link
Member Author

Also isn't checking installed? supposed to be unnecessary now?

Yeh, it should be.

I think that explains why you'd want to kill off keg_only :provided_until_xcode43 and :provided_until_xcode5 but why the others?

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.

@MikeMcQuaid
Copy link
Member Author

@Homebrew/core it may be worth checking the other uses of keg_only :provided_by_macos and keg_only :shadowed_by_macos to see if they all still apply on High Sierra 🤔.

@ilovezfs ilovezfs merged commit e27c4c3 into Homebrew:master Apr 17, 2018
@lock lock bot added the outdated PR was locked due to age label May 17, 2018
@lock lock bot locked as resolved and limited conversation to collaborators May 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants