drake 0.8.2.1.0.12 → 0.8.2.1.0.13

data/CHANGES.drake

@@ -1,6 +1,10 @@
 
 = Drake Changelog
 
+== Version 0.8.2.1.0.13
+
+* Optimized construction of computation tree.
+
 == Version 0.8.2.1.0.12
 
 * Handle multiple manual invocations of Task#invoke better.

data/lib/rake.rb

@@ -29,7 +29,7 @@
 # as a library via a require statement, but it can be distributed
 # independently as an application.
 
-RAKEVERSION = '0.8.2.1.0.12'
+RAKEVERSION = '0.8.2.1.0.13'
 
 require 'rbconfig'
 require 'fileutils'
@@ -582,7 +582,7 @@ module Rake
           application.parallel_tasks.clear
           application.parallel_parent_flags.clear
           base_invoke(*args)
-          application.invoke_parallel_tasks
+          application.invoke_parallel(self.name)
         }
       end
     end

data/lib/rake/parallel.rb

@@ -5,25 +5,10 @@ require 'rake/comp_tree/driver'
 
 module Rake
   module TaskManager
-    # :nodoc:
-    def invoke_parallel_tasks
-      parent_names = parallel_tasks.keys.map { |name|
-        name.to_sym
-      }
-
-      root_name = "computation_root__#{Process.pid}__#{rand}".to_sym
-         
+    def invoke_parallel(root_task_name) # :nodoc:
       CompTree::Driver.new(:discard_result => true) { |driver|
         #
-        # Define the root computation node.
-        #
-        # Top-level tasks are immediate children of the root.
-        #
-        driver.define(root_name, *parent_names) {
-        }
-
-        #
-        # build the rest of the computation tree from task prereqs
+        # Build the computation tree from task prereqs.
         #
         parallel_tasks.each_pair { |task_name, cache|
           task = self[task_name]
@@ -36,19 +21,29 @@ module Rake
           }
         }
 
+        root_node = driver.nodes[root_task_name.to_sym]
+
         #
-        # Mark computation nodes without a function as computed.
-        #
-        driver.nodes[root_name].each_downward { |node|
-          unless node.function
-            node.result = true
-          end
-        }
-        
-        #
-        # launch the computation
+        # If there were nothing to do, there would be no root node.
         #
-        driver.compute(root_name, :threads => num_threads, :fork => false)
+        if root_node
+          #
+          # Mark computation nodes without a function as computed.
+          #
+          root_node.each_downward { |node|
+            unless node.function
+              node.result = true
+            end
+          }
+
+          #
+          # Launch the computation.
+          #
+          driver.compute(
+            root_node.name,
+            :threads => num_threads,
+            :fork => false)
+        end
       }
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: drake
 version: !ruby/object:Gem::Version 
-  version: 0.8.2.1.0.12
+  version: 0.8.2.1.0.13
 platform: ruby
 authors: 
 - James M. Lawrence
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2008-09-19 00:00:00 -04:00
+date: 2008-09-20 00:00:00 -04:00
 default_executable: drake
 dependencies: []