rbbt-util 5.21.114 → 5.21.115

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0e4131497146d6065961b53d868e37e9646da540
-  data.tar.gz: bccbef978b8665d403f6ccab0d406cd4bbc9893f
+  metadata.gz: 646ea32ae9ce7ff64c1d2ba88179b9883ca19522
+  data.tar.gz: 7cfc05f02e007ce786ab4e2158fde5c324375ad3
 SHA512:
-  metadata.gz: 52627cf3091431622e0319ec71fa02f378a8b9ac530aa77f43bd969cacf446d4e98fa8c381a1a7c6357ae3bb91bdfc08162d7531d09c9947e83d1b37884af8b5
-  data.tar.gz: 2c5e96ee531bc7c0a73923c819c268b356709908d88dfa96f5ddae813d4cbeea3577e4d6414820a86c06e3cd54ba38540a940d0b906a3aa971bf1608b0ef0d52
+  metadata.gz: bf93e300a29a6736da94cd6f715e9a17088dfb6d1a5ac4b51315191d00668930ca53bb3cd8fbd162d0591b7f3173da3a5500fff55ca267f1c08702b77193293d
+  data.tar.gz: f143574d50f916ffa09e196335246e89fcdb99ef08ce82e22e17eb271bfefde22ab159f8b4a24befa1fe114911d162c4b39290878c33f5da7c89745773094c23

data/lib/rbbt/workflow/accessor.rb

@@ -146,7 +146,7 @@ class Step
     return nil if @exec or info_file.nil? or Open.exists?(info_file)
     Open.lock(info_file, :lock => info_lock) do
       i = {:status => :waiting, :pid => Process.pid}
-      i[:dependencies] = dependencies.collect{|dep| [dep.task_name, dep.name, dep.path]}
+      i[:dependencies] = dependencies.collect{|dep| [dep.task_name, dep.name, dep.path]} if dependencies
       @info_cache = i
       Misc.sensiblewrite(info_file, INFO_SERIALIAZER.dump(i), :force => true, :lock => false)
       @info_cache_time = Time.now
@@ -365,6 +365,15 @@ class Step
     Open.exists?(info_file) and not started?
   end
 
+  def dirty_files
+    dirty_files = rec_dependencies.reject{|dep|
+      (defined?(WorkflowRESTClient) && WorkflowRESTClient::RemoteStep === dep) || 
+        (dep.path && (Open.exists?(dep.path) || Open.remote?(dep.path))) || 
+        (dep.error? && ! dep.recoverable_error?)
+    }
+
+  end
+
   def dirty?
     return true if Open.exists?(pid_file) && ! ( Open.exists?(info_file) || done? )
     return false unless done? || status == :done
@@ -378,12 +387,6 @@ class Step
       return true 
     end
 
-    dirty_files = rec_dependencies.reject{|dep|
-      (defined?(WorkflowRESTClient) && WorkflowRESTClient::RemoteStep === dep) || 
-        (dep.path && (Open.exists?(dep.path) || Open.remote?(dep.path))) || 
-        (dep.error? && ! dep.recoverable_error?)
-    }
-
     if dirty_files.any?
       true
     else

data/lib/rbbt/workflow/step/run.rb

@@ -100,8 +100,8 @@ class Step
     (dependency_checks + input_checks).uniq
   end
 
-  def updated?
-    return true unless done?
+
+  def out_of_date
 
     outdated_time  = []
     outdated_dep  = []
@@ -117,7 +117,13 @@ class Step
     Log.low "Some newer files found: #{Misc.fingerprint outdated_time}" if outdated_time.any?
     Log.low "Some outdated files found: #{Misc.fingerprint outdated_dep}" if outdated_dep.any?
 
-    (outdated_time + outdated_dep).empty?
+    outdated_time + outdated_dep
+  end
+
+  def updated?
+    return true unless done?
+
+    out_of_date.empty?
   end
 
   def kill_children

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rbbt-util
 version: !ruby/object:Gem::Version
-  version: 5.21.114
+  version: 5.21.115
 platform: ruby
 authors:
 - Miguel Vazquez
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-12-12 00:00:00.000000000 Z
+date: 2017-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake