minisat: fails with clang

Closes Homebrew/legacy-homebrew#28778.
Homebrew · May 13, 2014 · c204651 · c204651
1 parent 54cfae5
commit c204651
Showing 1 changed file with 12 additions and 9 deletions.
diff --git a/Formula/minisat.rb b/Formula/minisat.rb
@@ -1,19 +1,22 @@
-require 'formula'
+require "formula"
 
 class Minisat < Formula
-  homepage 'http://minisat.se'
-  url 'https://github.com/niklasso/minisat/archive/releases/2.2.0.tar.gz'
-  sha1 '28c14eed485f4adb8dde9e26f05476f7eedc8f77'
+  homepage "http://minisat.se"
+  url "https://github.com/niklasso/minisat/archive/releases/2.2.0.tar.gz"
+  sha1 "28c14eed485f4adb8dde9e26f05476f7eedc8f77"
 
-  # Fixes compilation on mac; in next upstream release. See:
+  # Fix some declaration errors; see:
   # http://groups.google.com/group/minisat/browse_thread/thread/f5b6a180cadbb214
   patch :DATA
 
+  fails_with :clang do
+    cause "error: friend declaration specifying a default argument must be a definition"
+  end
+
   def install
-    ENV['MROOT'] = Dir.pwd
-    Dir.chdir 'simp'
-    system "make", "r"
-    bin.install 'minisat_release' => 'minisat'
+    ENV["MROOT"] = buildpath
+    system "make", "-C", "simp", "r"
+    bin.install "simp/minisat_release" => "minisat"
   end
 end