Always call make for FFI specs

* The fact that the library exists doesn't mean it's up to date.
* The Makefile does the right timestamp checks and compares to sources, and it's fast.
* Simplify code, `system` returns false or nil when the command fails.

1 files changed, 5 insertions, 8 deletions

diff --git a/spec/ffi/fixtures/compile.rb b/spec/ffi/fixtures/compile.rb
index 16892e0..6444601 100644
--- a/spec/ffi/fixtures/compile.rb
+++ b/spec/ffi/fixtures/compile.rb
@@ -50,15 +50,12 @@ module TestLibrary
   def self.compile_library(path, lib)
     dir = File.expand_path(path, File.dirname(__FILE__))
     lib = "#{dir}/#{lib}"
-    unless File.exist?(lib)
-      output = nil
-      FileUtils.cd(dir) do
-        make = ENV['MAKE'] || (system('which gmake >/dev/null') ? 'gmake' : 'make')
-        output = system(*%{#{make} CPU=#{CPU} OS=#{OS}}.tap{|x| puts x.inspect})
-      end
 
-      unless $?.success?
-        puts "ERROR:\n#{output}"
+    FileUtils.cd(dir) do
+      make = ENV['MAKE'] || (system('which gmake >/dev/null') ? 'gmake' : 'make')
+
+      unless system(*%{#{make} CPU=#{CPU} OS=#{OS}}.tap{ |cmd| puts cmd.inspect })
+        puts "ERROR: #{$?}"
         raise "Unable to compile #{lib.inspect}"
       end
     end