[Buildroot] [PATCH 1/1] bash: added option to disable locale support

Peter Korsgaard peter at korsgaard.com
Tue Mar 10 19:52:07 UTC 2015


>>>>> "Matthew" == Matthew Weber <matthew.weber at rockwellcollins.com> writes:

 > Thomas, all.
 > On Mon, Mar 9, 2015 at 4:42 PM, Thomas Petazzoni
 > <thomas.petazzoni at free-electrons.com> wrote:
 >> Dear Matt Weber,
 >> 
 >> On Mon,  9 Mar 2015 16:13:15 -0500, Matt Weber wrote:
 >> 
 >>> +if BR2_PACKAGE_BASH
 >>> +
 >>> +config BR2_PACKAGE_BASH_NO_LOCALE
 >>> +     bool "disable locale support"
 >>> +     help
 >>> +       Disable locale support for cases where toolchain has it
 >>> +       but target isn't setup to use it.
 >>> +
 >>> +endif
 >> 
 >> To be honest, I'm not sure we want to support such cases. What does it
 >> mean for the target to not be setup to use locales?

 > I should probably clarify, this disabled the runtime check of locale
 > that bash did everytime a shell was invoked.  All packages where still
 > built with locale support.  Busybox ash didn't perform this check so
 > we never noticed it until we switched to bash.  (I believe, will have
 > to double check)Bash is running  setlocale after checking for a env
 > variable LC_xxxxx before the shell starts.  Currently we don't have
 > our env setup to by default have LC_xxxxx set.  Instead we  set the
 > LC_xxxxx as part of the service/app start rcS script. (There maybe
 > better way's of handling this I'm sure).

So what is the impact exactly of having/not having this option?

-- 
Bye, Peter Korsgaard



More information about the buildroot mailing list