[Pkg-puppet-devel] [SCM] Puppet packaging for Debian branch, upstream, updated. 0.25.4-89-gcbbd363

[James Turnbull]

The following commit has been merged in the upstream branch:
commit f9e05a8062423cf0e4dd6dca2050a8c7d4b2e85d
Author: Markus Roberts <Markus at reality.com>
Date:   Fri Jan 22 12:55:14 2010 -0800

    Fix for #3094 (libdir should take ":" delimited path)
    
    Actually, File::PATH_SEPARATOR, which is generally, but not always, ":").
    
    Since libdir is also the default for the plugin handler, users will need to
    specify it explicitly if a multipart libdir is given (and it will need to be
    one of the segments given in the libdir for the plugins to be found).

diff --git a/lib/puppet/util/autoload.rb b/lib/puppet/util/autoload.rb
index 51fdaad..ceaabe4 100644
--- a/lib/puppet/util/autoload.rb
+++ b/lib/puppet/util/autoload.rb
@@ -156,7 +156,7 @@ class Puppet::Util::Autoload
         end
     end
 
-    def search_directories
-        [module_directories, Puppet[:libdir], $:].flatten
+    def search_directories(dummy_argument=:work_arround_for_ruby_GC_bug)
+        [module_directories, Puppet[:libdir].split(File::PATH_SEPARATOR), $:].flatten
     end
 end
diff --git a/spec/unit/util/autoload.rb b/spec/unit/util/autoload.rb
index 4e13842..220cb5f 100755
--- a/spec/unit/util/autoload.rb
+++ b/spec/unit/util/autoload.rb
@@ -51,8 +51,9 @@ describe Puppet::Util::Autoload do
         end
 
         it "should include the module directories, the Puppet libdir, and all of the Ruby load directories" do
+            Puppet.stubs(:[]).with(:libdir).returns(%w{/libdir1 /lib/dir/two /third/lib/dir}.join(File::PATH_SEPARATOR))
             @autoload.expects(:module_directories).returns %w{/one /two}
-            @autoload.search_directories.should == ["/one", "/two", Puppet[:libdir], $:].flatten
+            @autoload.search_directories.should == %w{/one /two /libdir1 /lib/dir/two /third/lib/dir} + $:
         end
 
         it "should include in its search path all of the search directories that have a subdirectory matching the autoload path" do

-- 
Puppet packaging for Debian