Puppet: System Administration Automated

Support

Ticket #1007: exec.rb.patch

File exec.rb.patch, 2.4 kB (added by Fujin, 11 months ago)

path for exec.rb to change env to environment

  • a/lib/puppet/type/exec.rb

    old new  
    228228            end 
    229229        end 
    230230 
    231         newparam(:env) do 
     231        newparam(:environment) do 
    232232            desc "Any additional environment variables you want to set for a 
    233233                command.  Note that if you use this to set PATH, it will override 
    234234                the ``path`` attribute.  Multiple environment variables should be 
    …… 
    554554            begin 
    555555                # Do our chdir 
    556556                Dir.chdir(dir) do 
    557                     env = {} 
     557                    environment = {} 
    558558 
    559559                    if self[:path] 
    560                         env[:PATH] = self[:path].join(":") 
     560                        environment[:PATH] = self[:path].join(":") 
    561561                    end 
    562562 
    563                     if envlist = self[:env] 
     563                    if envlist = self[:environment] 
    564564                        envlist = [envlist] unless envlist.is_a? Array 
    565565                        envlist.each do |setting| 
    566566                            if setting =~ /^(\w+)=((.|\n)+)$/ 
    567567                                name = $1 
    568568                                value = $2 
    569                                 if env.include? name 
     569                                if environment.include? name 
    570570                                    warning( 
    571571                                    "Overriding environment setting '%s' with '%s'" % 
    572572                                        [name, value] 
    573573                                    ) 
    574574                                end 
    575                                 env[name] = value 
     575                                environment[name] = value 
    576576                            else 
    577                                 warning "Cannot understand env setting %s" % setting.inspect 
     577                                warning "Cannot understand environment setting %s" % setting.inspect 
    578578                            end 
    579579                        end 
    580580                    end 
    581581 
    582                     withenv env do 
     582                    withenv environment do 
    583583                        Timeout::timeout(self[:timeout]) do 
    584584                            output, status = Puppet::Util::SUIDManager.run_and_capture( 
    585585                                [command], self[:user], self[:group]