Issue #4695 has been updated by Luke Kanies.

The original commit is commit:"79dcd33a", and it specifies that the locale is 
changed to make output easier to parse.

Maybe we should make this an option?

My guess is that not changing locale will cause some providers to break, given 
how much they rely on parsing output of commands.  How could we test this?  Or 
should we change providers to default to setting locale et al just on their 
commands?
----------------------------------------
Bug #4695: Puppet should not reset locale
http://projects.puppetlabs.com/issues/4695

Author: Mo Morsi
Status: Needs design decision
Priority: Normal
Assignee: Luke Kanies
Category: exec
Target version: 
Affected version: 
Keywords: 
Branch: 


Puppet resets various locale related ENV variables to the posix default of 'C' 
immediately before invoked Kernel.exec to launch an executable (occurs when 
running a Single_exec in a recipe)

http://github.com/reductivelabs/puppet/blob/master/lib/puppet/util.rb#L295

The problem is, the python interpreter will attempt to parse these variables to 
determine the default locale, and if it is set to 'C' (or not set at all as it 
will default to 'C'), 'None' will be returned, ignoring the true default locale.

Thus any python script launched via Single_exec from a puppet recipe will not 
be able to get the default locale by invoking locale.getdefaultlocale(). This 
functionality is part of the standard python library and is relied upon by many 
various Python scripts.


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