<?xml version="1.0" encoding="UTF-8"?>
<commit>
  <added type="array"/>
  <modified type="array">
    <modified>
      <diff>@@ -11,6 +11,7 @@ require &quot;dm-validations&quot;
 require &quot;dm-types&quot;
 require &quot;dm-timestamps&quot;
 require &quot;dm-aggregates&quot;
+require &quot;open4&quot;
 
 require &quot;yaml&quot;
 require &quot;logger&quot;</diff>
      <filename>lib/integrity.rb</filename>
    </modified>
    <modified>
      <diff>@@ -1,4 +1,4 @@
-module Integrity
+module Integrity  
   class Build
     include DataMapper::Resource
 
@@ -16,7 +16,7 @@ module Integrity
     has 1,     :commit
 
     before :destroy do commit.destroy! end
-    after :save, :run_onsuccess
+    before :save, :run_onsuccess
 
     def self.pending
       all(:started_at =&gt; nil)
@@ -55,14 +55,13 @@ module Integrity
     private
 
     def run_onsuccess
-      if successful? and !project.onsuccess.nil? and !project.onsuccess.empty?
+      if successful? and project.onsuccess
         pid, stdin, stdout, stderr = Open4::popen4 &quot;#{project.onsuccess}&quot;
-        if stderr.empty?
-          self.onsuccess_output = stdout
+        if stderr.read.strip.empty?
+          onsuccess_output = stdout.read.strip
         else
-          self.onsuccess_output = stderr
+          onsuccess_output = stderr.read.strip
         end
-        self.save
       end
     end
   end</diff>
      <filename>lib/integrity/build.rb</filename>
    </modified>
    <modified>
      <diff>@@ -27,4 +27,4 @@
   %h2 Success Output:
   %pre.output
     :preserve
-      #{bash_color_codes h(build.onsuccess_output)}
\ No newline at end of file
+      #{bash_color_codes h(build.onsuccess_output)}</diff>
      <filename>views/_build_info.haml</filename>
    </modified>
  </modified>
  <removed type="array"/>
  <parents type="array">
    <parent>
      <id>67f9aff5cb831c36e8b5153404d8f15b29ebd638</id>
    </parent>
  </parents>
  <author>
    <name>Michael Erb</name>
    <email>michael@erbmicha.com</email>
  </author>
  <url>http://github.com/erbmicha/integrity/commit/416c05498efe421e24dcc724d3efe89007f3203e</url>
  <id>416c05498efe421e24dcc724d3efe89007f3203e</id>
  <committed-date>2009-11-05T09:39:16-08:00</committed-date>
  <authored-date>2009-11-05T09:39:16-08:00</authored-date>
  <message>added require for open4 to integrity.rb (what a dummy...)
running :run_onsuccess before save now to avoid that nice infinite loop (moron)
added blank newline at the end of build info partial just in case...</message>
  <tree>8bb4e6c823cda38688f1678e6236dc6b9debf747</tree>
  <committer>
    <name>Michael Erb</name>
    <email>michael@erbmicha.com</email>
  </committer>
</commit>
