brew-audit: Warn against depending on MPI

Depending on `open-mpi` or `mpich2` will cause problems since both formulae
install components with the same names. `brew audit` now recommends using
MPIDependency.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>

1 files changed, 8 insertions, 0 deletions

diff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb
index 0fd44580b..5f7641a33 100755
--- a/Library/Homebrew/cmd/audit.rb
+++ b/Library/Homebrew/cmd/audit.rb
@@ -337,6 +337,14 @@ def audit_formula_instance f
 EOS
     when 'gfortran'
       problems << " * Use ENV.fortran during install instead of depends_on 'gfortran'"
+
+    when 'open-mpi', 'mpich2'
+      problems << <<-EOS.undent
+          * There are multiple conflicting ways to install MPI. Use a MPIDependency:
+              depends_on MPIDependency.new(<lang list>)
+            Where <lang list> is a comma delimited list that can include:
+              :cc, :cxx, :f90, :f77
+        EOS
     end
   end