aboutsummaryrefslogtreecommitdiffstats
path: root/Library
diff options
context:
space:
mode:
authorJack Nagel2014-10-17 00:35:51 -0500
committerJack Nagel2014-10-17 00:35:51 -0500
commitb6631b9a150250c2f381912d3bd360ae1d72974c (patch)
tree8bded5e5f3eae39a20b944fc0a3a94a9f56b7a58 /Library
parentfb30041640852f14771ac928f655fb3d17a33926 (diff)
downloadbrew-b6631b9a150250c2f381912d3bd360ae1d72974c.tar.bz2
audit: call puts once instead of problems.size + 2 times
Diffstat (limited to 'Library')
-rw-r--r--Library/Homebrew/cmd/audit.rb9
1 files changed, 2 insertions, 7 deletions
diff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb
index a0359ae07..4b9aba93c 100644
--- a/Library/Homebrew/cmd/audit.rb
+++ b/Library/Homebrew/cmd/audit.rb
@@ -22,11 +22,9 @@ module Homebrew
fa.audit
unless fa.problems.empty?
- puts "#{f.name}:"
- fa.problems.each { |p| puts " * #{p}" }
- puts
formula_count += 1
problem_count += fa.problems.size
+ puts "#{f.name}:", fa.problems.map { |p| " * #{p}" }, ""
end
end
@@ -153,10 +151,7 @@ class FormulaAuditor
next if dep.build? or dep.run?
problem %{#{dep} dependency should be "depends_on '#{dep}' => :build"}
when "git", "ruby", "mercurial"
- problem <<-EOS.undent
- Don't use #{dep} as a dependency. We allow non-Homebrew
- #{dep} installations.
- EOS
+ problem "Don't use #{dep} as a dependency. We allow non-Homebrew #{dep} installations."
when 'gfortran'
problem "Use `depends_on :fortran` instead of `depends_on 'gfortran'`"
when 'open-mpi', 'mpich2'