0
@@ -1 +1,2 @@
0
 *.swp
0
+.cache.yaml

0
@@ -2,6 +2,8 @@ require 'logging'
0
 
0
 module Brake
0
   module Checks
0
+    class CheckException < Exception; end
0
+
0
     include Brake::Logging
0
     
0
     def find_program(program_name, paths = [])
0
@@ -32,18 +34,25 @@ module Brake
0
     end
0
 
0
     def try_compile_and_run(name, source_code, ext, compiler)
0
-        provide_temp_directory
0
+      provide_temp_directory
0
 
0
-        source = "./tmp/try_#{name}_#{$$}.#{ext}"
0
-        target = "./tmp/try_#{name}_#{$$}"
0
+      source = "./tmp/try_#{name}_#{$$}.#{ext}"
0
+      target = "./tmp/try_#{name}_#{$$}"
0
 
0
-        File.open(source, "w") do |f|
0
-          f.write(source_code)
0
-        end
0
+      File.open(source, "w") do |f|
0
+        f.write(source_code)
0
+      end
0
 
0
-        compiler.oneliner(source, target)
0
+      raise CheckException, "Compiler returned an error" unless compiler.oneliner(source, target)
0
+      raise CheckException, "Compiler stated success, but no target was found" unless File.exists? target
0
 
0
-        File.unlink(source)
0
+    rescue CheckException
0
+      return false
0
+    else
0
+      return true
0
+    ensure
0
+      File.unlink(source) if File.exists? source
0
+      File.unlink(target) if File.exists? target
0
     end
0
 
0
     private

0
@@ -21,12 +21,11 @@ module Brake
0
         raise GccException, "Compiler didn't produce a working program" 
0
           unless try_compile_and_run("int main() { return 0; }\n", "c", self)
0
 
0
-        rescue => e
0
-          log e.message, :fatal
0
-
0
-          return false
0
-        end
0
+      rescue => e
0
+        log e.message, :fatal
0
 
0
+        false
0
+      else
0
         true
0
       end
0
     end

0
@@ -74,12 +74,12 @@ describe "try_compile_and_run" do
0
   end
0
   
0
   it "should set proper extension for source" do 
0
-    @compiler.should_receive(:oneliner).with(/\.c$/, an_instance_of(String))
0
+    @compiler.should_receive(:oneliner).with(/\.c$/, an_instance_of(String)).once.and_return(true)
0
     
0
     try_compile_and_run('name', '', 'c', @compiler)
0
     
0
     compiler2 = mock('compiler')
0
-    compiler2.should_receive(:oneliner).with(/\.qqq$/, an_instance_of(String))
0
+    compiler2.should_receive(:oneliner).with(/\.qqq$/, an_instance_of(String)).once.and_return(true)
0
     
0
     try_compile_and_run('name', '', 'qqq', compiler2)
0
   end
0
@@ -100,12 +100,13 @@ describe "try_compile_and_run" do
0
     SOURCE_CODE = "/* some code */\nint main() { return 0; }\n"
0
     @compiler.should_receive(:oneliner) do |source, target|
0
       IO.read(source).should == SOURCE_CODE 
0
+      true
0
     end
0
 
0
     try_compile_and_run('name', SOURCE_CODE, 'c', @compiler)
0
   end
0
 
0
-  it "should remove generated source code upon return" do
0
+  it "should remove generated files upon return" do
0
     @compiler.should_receive(:oneliner) do |source, target|
0
       @source = source
0
       @target = target
0
@@ -118,4 +119,16 @@ describe "try_compile_and_run" do
0
     File.exists?(@source).should_not == true
0
     File.exists?(@target).should_not == true
0
   end
0
+
0
+  it "should return false if compiler failed" do
0
+    @compiler.should_receive(:oneliner).once.and_return(false)
0
+
0
+    try_compile_and_run('name', '', 'c', @compiler).should == false
0
+  end
0
+
0
+  it "should return false if compiler succeeded, but no target found" do
0
+    @compiler.should_receive(:oneliner).once.and_return(true)
0
+
0
+    try_compile_and_run('name', '', 'c', @compiler).should == false
0
+  end
0
 end