build.rb in damagecontrol-0.5.0.1391

- old
+ new

@@ -1,11 +1,14 @@
 require 'rscm/path_converter'
 require 'damagecontrol/directories'
+require 'damagecontrol/project'
 
 module DamageControl
-  # File structure
+  # Represents build-related data organised in the following file structure:
   #
+  # File structure:
+  #
   #   .damagecontrol/
   #     SomeProject/
   #       project.yaml
   #       checkout/
   #       changesets/
@@ -17,20 +20,36 @@
   #               stdout.log
   #               stderr.log
   #               artifacts/
   #
   class Build
+
+    # TODO: we want to store the following additional info for a build (time related)
+    #  * Total Duration
+    #  * Duration of checkpoints (compile, test, javadocs...) - should be configurable in project.
+    #  * 
+
     attr_reader :time
   
     # Creates a new Build for a +project+'s +changeset+, created at +time+.
-    def initialize(project_name, changeset_identifier, time)
-      @project_name, @changeset_identifier, @time = project_name, changeset_identifier, time
+    def initialize(project_name, changeset_identifier, time, build_reason)
+      @project_name, @changeset_identifier, @time, @build_reason = project_name, changeset_identifier, time, build_reason
     end
 
-    # The changeset we belong to
+    # Our unique id within the changeset
+    def identifier
+      time.ymdHMS
+    end
+
+    # Our associated project
+    def project
+      Project.load(@project_name)
+    end
+    
+    # Our associated changeset
     def changeset
-      Directories.changeset
+      project.changeset(@changeset_identifier)
     end
     
     # Executes +command+ with the environment variables +env+ and persists the command for future reference.
     # This will prevent the same build from being executed in the future.
     def execute(command, env={})
@@ -38,24 +57,26 @@
       raise BuildException.new("This build has already been executed and cannot be re-executed. It was executed with '#{File.open(command_file).read}'") if File.exist?(command_file)
       FileUtils.mkdir_p(File.dirname(command_file))
       File.open(command_file, "w") do |io|
         io.write(command)
       end
-      stderr = Directories.stderr(@project_name, @changeset_identifier, @time)
-      stdout = Directories.stdout(@project_name, @changeset_identifier, @time)
       command_line = "#{command} > #{stdout} 2> #{stderr}"
 
       begin
         with_working_dir(checkout_dir) do
           env.each {|k,v| ENV[k]=v}
+          Log.info "Executing '#{command_line}'"
+          Log.info "Execution environment:"
+          ENV.each {|k,v| Log.info("#{k}=#{v}")}
           IO.popen(command_line) do |io|
             File.open(pid_file, "w") do |pid_io|
               pid_io.write(pid)
             end
             
             # there is nothing to read, since we're redirecting to file,
-            # but we still need to read in order to block till process id done.
+            # but we still need to read in order to block until the process is done.
+            # TODO: don't redirect stdout - we want to intercept checkpoints
             io.read
           end
         end
       ensure
         exit_code = $? >> 8
@@ -71,19 +92,40 @@
         File.read(exit_code_file).to_i
       else
         nil
       end
     end
+    
+    def successful?
+      exit_code == 0
+    end
+    
+    def status_message
+      successful? ? "Successful" : "Failed"
+    end
 
     # Returns the pid of the build process
     def pid
       File.read(pid_file).to_i
     end
     
     def kill
       Process.kill("SIGHUP", pid)
     end
+
+    def stdout
+      Directories.stdout(@project_name, @changeset_identifier, @time)
+    end
+
+    def stderr
+      Directories.stderr(@project_name, @changeset_identifier, @time)
+    end
     
+    # The directory of the build
+    def dir
+      Directories.build_dir(@project_name, @changeset_identifier, @time)
+    end
+
   private
   
     def checkout_dir
       Directories.checkout_dir(@project_name)
     end
\ No newline at end of file