Skip to content

Commit

Permalink
Remove macOS implementation of #arch_compatible?
Browse files Browse the repository at this point in the history
We don't really need this.
  • Loading branch information
carlocab committed Sep 30, 2024
1 parent 861d7b9 commit 6329db9
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 8 deletions.
2 changes: 1 addition & 1 deletion Library/Homebrew/extend/pathname.rb
Original file line number Diff line number Diff line change
Expand Up @@ -471,7 +471,7 @@ def dylib?

sig { params(_wanted_arch: Symbol).returns(T::Boolean) }
def arch_compatible?(_wanted_arch)
false
true

Check warning on line 474 in Library/Homebrew/extend/pathname.rb

View check run for this annotation

Codecov / codecov/patch

Library/Homebrew/extend/pathname.rb#L474

Added line #L474 was not covered by tests
end

sig { returns(T::Array[String]) }
Expand Down
7 changes: 0 additions & 7 deletions Library/Homebrew/os/mac/mach.rb
Original file line number Diff line number Diff line change
Expand Up @@ -155,13 +155,6 @@ def ppc64?
arch == :ppc64
end

def arch_compatible?(wanted_arch)
return true unless mach_data.present?
return arch == wanted_arch unless universal?

archs.include?(wanted_arch)
end

def dylib?
mach_data.any? { |m| m.fetch(:type) == :dylib }
end
Expand Down

0 comments on commit 6329db9

Please sign in to comment.