diff options
| author | Paul Lathrop <paul@tertiusfamily.net> | 2008-01-21 17:01:53 -0800 |
|---|---|---|
| committer | Paul Lathrop <paul@tertiusfamily.net> | 2008-02-05 13:13:40 -0800 |
| commit | 139ff33e1f93a0634547fc9a06442a720fe0a58e (patch) | |
| tree | e63e42e0b366f09324b3895bde3b687b3aae9347 /lib/puppet | |
| parent | 1b1164d5d614b0d2c37b058edb6a126c7fd0bae2 (diff) | |
| download | puppet-139ff33e1f93a0634547fc9a06442a720fe0a58e.tar.gz puppet-139ff33e1f93a0634547fc9a06442a720fe0a58e.tar.xz puppet-139ff33e1f93a0634547fc9a06442a720fe0a58e.zip | |
Fujin's patch for ticket #1007 - consistent use of 'environment' instead of 'env'
Diffstat (limited to 'lib/puppet')
| -rwxr-xr-x | lib/puppet/type/exec.rb | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/lib/puppet/type/exec.rb b/lib/puppet/type/exec.rb index 5bb3158c4..a7a046c43 100755 --- a/lib/puppet/type/exec.rb +++ b/lib/puppet/type/exec.rb @@ -228,7 +228,7 @@ module Puppet end end - newparam(:env) do + newparam(:environment) do desc "Any additional environment variables you want to set for a command. Note that if you use this to set PATH, it will override the ``path`` attribute. Multiple environment variables should be @@ -554,32 +554,32 @@ module Puppet begin # Do our chdir Dir.chdir(dir) do - env = {} + environment = {} if self[:path] - env[:PATH] = self[:path].join(":") + environment[:PATH] = self[:path].join(":") end - if envlist = self[:env] + if envlist = self[:environment] envlist = [envlist] unless envlist.is_a? Array envlist.each do |setting| if setting =~ /^(\w+)=((.|\n)+)$/ name = $1 value = $2 - if env.include? name + if environment.include? name warning( "Overriding environment setting '%s' with '%s'" % [name, value] ) end - env[name] = value + environment[name] = value else - warning "Cannot understand env setting %s" % setting.inspect + warning "Cannot understand environment setting %s" % setting.inspect end end end - withenv env do + withenv environment do Timeout::timeout(self[:timeout]) do output, status = Puppet::Util::SUIDManager.run_and_capture( [command], self[:user], self[:group] |
