Issue #3411 has been updated by Jos Backus.

How about the following patch? If the user set TMP or TEMP, why not assume they 
meant for us to use them? That way the issue mentioned can be solved simply by 
setting one of these environment variables appropriately.

             opts.parse!e isues
         end
    -    tmpdirs = [".", ENV['TMP'], ENV['TEMP'], "/tmp", "/var/tmp"]
    +    tmpdirs = [ENV['TMP'], ENV['TEMP'], ".", "/tmp", "/var/tmp"]
    
         version = [Config::CONFIG["MAJOR"], Config::CONFIG["MINOR"]].join(".")
         libdir = File.join(Config::CONFIG["libdir"], "ruby", version)

----------------------------------------
Bug #3411: install.rb should not put "." first in the tmp_dirs
http://projects.puppetlabs.com/issues/3411

Author: Martin Englund
Status: Investigating
Priority: Normal
Assigned to: Martin Englund
Category: 
Target version: 
Keywords: 
Branch: 


Since Ruby incorrectly assumes that being root is enough to be able to write a 
directory, install.rb will fail when installing as root from a NFS mounted 
directory, as it will pick the current directory (".") as tmp_dir.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to