lib/build-tool/model/module.rb in build-tool-0.6.7 vs lib/build-tool/model/module.rb in build-tool-0.6.8

- old
+ new

@@ -84,10 +84,17 @@ else return active end end + def dirty? + if vcs + return vcs.dirty? + end + return false + end + # not inherited def build_directory build_prefix_required.join("bld", local_path) end @@ -263,16 +270,21 @@ return BuildTool::History::CommandLog.last_by_module( name ) end # Returns a current state in string format def state - return 'UNKNOWN' if lastlog.empty? - lastlog[0].module_logs.where( :module => name ).each do |e| - if e.state != History::ModuleLog::FINISHED_SUCCESSFUL - return "#{e.state_str} (#{e.event})" - end + log = lastlog + return 'UNKNOWN' if log.empty? + + e = log[0].module_logs.reverse_order.where( :module => name ).first + if e.nil? + return "UNKNOWN" end - return History::ModuleLog::state_str( History::ModuleLog::FINISHED_SUCCESSFUL ) + if e.state != History::ModuleLog::FINISHED_SUCCESSFUL + return "#{e.started_at.strftime("%x %X")}: #{e.state_str} (#{e.event})" + else + return "#{e.started_at.strftime("%x %X")}: #{e.state_str}" + end end # Return the current state as one char. def state_char return '?' if lastlog.empty?