artem-sidorenko / chef-cups

Chef cookbook for CUPS management
Apache License 2.0
2 stars 9 forks source link

Use an array instead of string for command #15

Closed fcoelho closed 8 years ago

fcoelho commented 8 years ago

Using an array makes it unecessary to escape strings for arguments, as the child process receives each element in the string as a separate argument regardless of its contents.

I have to use printers that are accessible via USB and have URIs with characters such as "&". Instead of trying to quote everything, this just sends the strings as arguments directly, no quoting needed anywhere.

jking916 commented 8 years ago

All tests passed on my end. Merging this in.