Issue #2193 has been updated by Markus Roberts.

At the very least the description on the ticket is no longer accurate; we 
require an array (not a string) in lib/puppet/util.rb's execute.  I suspect 
that this was the case at the time the patch was written as well.


----------------------------------------
Bug #2193: Exec's "command" parameter should accept lists in addition to strings
http://projects.puppetlabs.com/issues/2193

Author: Deepak Giridharagopal
Status: Accepted
Priority: Normal
Assigned to: Markus Roberts
Category: exec
Target version: Statler
Affected version: 0.25.0
Keywords: 
Branch: 


Currently, "exec" executes its "command" by passing a string to Kernel#exec, 
which subjects the command to shell-expansion.

There are many use cases where the user knows the exact command they wish to 
execute, and does not want the shell involved at all in the execution (say, for 
example, you're trying to pass an argument to a command that has weird 
characters in it that the shell would normally expand out).

Exec's "command" param should accept a list as input, where the list follows 
the format accepted by Kernel#exec ("If multiple arguments are given, the 
second and subsequent arguments are passed as parameters to command with no 
shell expansion.").

I've attached a patch against 0.25 HEAD (minimally tested)...hopefully that can 
be a starting point for development.


-- 
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