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
rubocop: Formulae with the "lxml" resource have required dependencies #16705
rubocop: Formulae with the "lxml" resource have required dependencies #16705
Conversation
- We're moving from `depends_on "python-lxml"` to `resource "lxml" ...` as part of the new Python vendoring plan. - For `resource "lxml"` to work, `uses_from_macos "libxml2"` and `uses_from_macos "libxslt"` are needed in the formulae. - This new RuboCop rule enforces that a formula including the "lxml" resource also include those dependencies.
Thank you so much for this! We have already moved some formulae over to using the resource, and will be moving the rest over as time allows. |
Thanks! I was also thinking that this would be very useful to just enforce. |
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.
This is nice. Thanks @issyl0!
Library/Homebrew/test/rubocops/resource_requires_dependencies_spec.rb
Outdated
Show resolved
Hide resolved
Library/Homebrew/test/rubocops/resource_requires_dependencies_spec.rb
Outdated
Show resolved
Hide resolved
Co-authored-by: Ruoyu Zhong <zhongruoyu@outlook.com>
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.
Great work as usual @issyl0!
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?depends_on "python-lxml"
toresource "lxml" ...
as part of the new Python vendoring plan.resource "lxml"
to work,uses_from_macos "libxml2"
anduses_from_macos "libxslt"
are needed in the formulae.TODO:
depends_on "python-lxml"
). Separate PR?