diff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb index f05fe8b077..10b37e57a6 100644 --- a/Library/Homebrew/cmd/audit.rb +++ b/Library/Homebrew/cmd/audit.rb @@ -151,11 +151,7 @@ class FormulaAuditor case dep.name when *BUILD_TIME_DEPS - next if dep.build? - next if dep.name == 'autoconf' && f.name =~ /automake/ - next if dep.name == 'libtool' && %w{imagemagick libgphoto2 libp11 libextractor}.any? { |n| f.name == n } - next if dep.name =~ /autoconf|pkg-config/ && f.name == 'ruby-build' - + next if dep.build? or dep.run? problem %{#{dep} dependency should be "depends_on '#{dep}' => :build"} when "git", "ruby", "emacs", "mercurial" problem <<-EOS.undent diff --git a/Library/Homebrew/dependency_collector.rb b/Library/Homebrew/dependency_collector.rb index 43e7010783..8ede0c1448 100644 --- a/Library/Homebrew/dependency_collector.rb +++ b/Library/Homebrew/dependency_collector.rb @@ -132,14 +132,15 @@ class DependencyCollector end def autotools_dep(spec, tags) - unless MacOS::Xcode.provides_autotools? - case spec - when :libltdl then spec = :libtool - else tags << :build - end + return if MacOS::Xcode.provides_autotools? - Dependency.new(spec.to_s, tags) + if spec == :libltdl + spec = :libtool + tags << :run end + + tags << :build unless tags.include? :run + Dependency.new(spec.to_s, tags) end def ant_dep(spec, tags)