admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Merge pull request #17032 from cho-m/audit-allow-lxml-build-resource

Browse Source 
rubocops/resource_requires_dependencies: allow `lxml` build resource
This commit is contained in:
Mike McQuaid 2024-04-05 13:26:31 +01:00 committed by GitHub
commit ad196e057b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
Library/Homebrew/rubocops/resource_requires_dependencies.rb
View File

@ -23,14 +23,20 @@ module RuboCop
next unless found
uses_from_macos_nodes = find_every_method_call_by_name(body_node, :uses_from_macos)
uses_from_macos = uses_from_macos_nodes.map { |node| node.arguments.first.str_content }
depends_on_nodes = find_every_method_call_by_name(body_node, :depends_on)
depends_on = depends_on_nodes.map { |node| node.arguments.first.str_content }
uses_from_macos_or_depends_on = (uses_from_macos_nodes + depends_on_nodes).filter_map do |node|
if (dep = node.arguments.first).hash_type?
dep_types = dep.values.first
dep_types = dep_types.array_type? ? dep_types.values.map(&:value) : [dep_types.value]
dep.keys.first.str_content if dep_types.include?(:build)
else
dep.str_content
end
end
required_deps = case resource_name
when "lxml"
kind = "uses_from_macos"
kind = depends_on?(:linux) ? "depends_on" : "uses_from_macos"
["libxml2", "libxslt"]
when "pyyaml"
kind = "depends_on"
@ -38,7 +44,7 @@ module RuboCop
else
[]
end
next if required_deps.all? { |dep| uses_from_macos.include?(dep) || depends_on.include?(dep) }
next if required_deps.all? { |dep| uses_from_macos_or_depends_on.include?(dep) }
offending_node(found)
problem "Add `#{kind}` lines above for #{required_deps.map { |req| "`\"#{req}\"`" }.join(" and ")}."