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

requirements: move more to compat. #3612

Merged
merged 4 commits into from
Dec 31, 2017
Merged

requirements: move more to compat. #3612

merged 4 commits into from
Dec 31, 2017

Conversation

MikeMcQuaid
Copy link
Member

@MikeMcQuaid MikeMcQuaid commented Dec 30, 2017

Also, while we're here:

  • gpg: move to compat.
  • lines_cop: check for more deprecated requirements.
  • lines_cop: deprecate 'needs :openmp'

@MikeMcQuaid MikeMcQuaid mentioned this pull request Dec 30, 2017
4 tasks
@DomT4
Copy link
Contributor

DomT4 commented Jan 3, 2018

I missed the perl bit here.

Noting that this isn't disagreement or looking for an argument, a commitment to using Homebrew's perl is a commitment to revision bumping practically everything that brew uses perl on every version bump to the perl formula even if it's a minor. Perl libraries get installed in lib/perl5/#{version} and lib/perl5/site_perl/#{version}, so you can see the issue.

Happens with the system perl too but obviously that's almost never updated.

@ilovezfs
Copy link
Contributor

ilovezfs commented Jan 3, 2018

Where system perl works, we can use it.

@MikeMcQuaid
Copy link
Member Author

This already happens where perl is the default_formula. These deprecations are to reduce the deviations between building from source and building/installing bottles.

@DomT4
Copy link
Contributor

DomT4 commented Jan 3, 2018

Where system perl works, we can use it.

Thanks, that's the clarification I was looking for.

@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants