Re: [PATCH] maint: set LANG=C instead of LC_ALL=C

2017-08-11 Thread Bruno Haible
Eric Blake wrote: > Still, if we are advocating mixed locale > execution, we MUST ensure sane defaults for ALL of the LC_* variables. In theory all the LC_* variables, yes. In practice, only those locale categories that the programs actually uses (e.g. test-quotearg.c). Which rarely goes beyond LC

Re: [PATCH] maint: set LANG=C instead of LC_ALL=C

2017-08-10 Thread Eric Blake
On 08/10/2017 05:37 PM, Bruno Haible wrote: >> You still want a sane fallback for all the categories that you are not >> explicitly setting. It's harder to type: >> >> LANG=C LC_CTYPE=C.UTF-8 env -u LC_ALL foo > > or: > LANG=C LC_CTYPE=C.UTF-8 LC_ALL= foo If setting LC_ALL to empty forces fall

Re: [PATCH] maint: set LANG=C instead of LC_ALL=C

2017-08-10 Thread Bruno Haible
> You still want a sane fallback for all the categories that you are not > explicitly setting. It's harder to type: > > LANG=C LC_CTYPE=C.UTF-8 env -u LC_ALL foo or: LANG=C LC_CTYPE=C.UTF-8 LC_ALL= foo > than it is to type > > LC_CTYPE=C.UTF-8 foo > > where we know that LANG=C is already se

Re: [PATCH] maint: set LANG=C instead of LC_ALL=C

2017-08-10 Thread Eric Blake
On 08/10/2017 11:36 AM, Daniel P. Berrange wrote: > The maint.mk file currently sets LC_ALL=C so that build rules get a > predictable locale, independant of the user's environment settings. s/independant/independent/ > > It is sometimes neccesssary to override the locale when running s/neccesss

Re: [PATCH] maint: set LANG=C instead of LC_ALL=C

2017-08-10 Thread Eric Blake
On 08/10/2017 12:59 PM, Paul Eggert wrote: > Daniel P. Berrange wrote: >> It is sometimes neccesssary to override the locale when running >> build commands from make rules, but as maint.mk set LC_ALL, it >> is impossible to selectively override rules e.g. LC_CTYPE=C.UTF-8 >> will have no effect if

Re: [PATCH] maint: set LANG=C instead of LC_ALL=C

2017-08-10 Thread Paul Eggert
Daniel P. Berrange wrote: It is sometimes neccesssary to override the locale when running build commands from make rules, but as maint.mk set LC_ALL, it is impossible to selectively override rules e.g. LC_CTYPE=C.UTF-8 will have no effect if LC_ALL is already set. Why not unset LC_ALL?

[PATCH] maint: set LANG=C instead of LC_ALL=C

2017-08-10 Thread Daniel P. Berrange
The maint.mk file currently sets LC_ALL=C so that build rules get a predictable locale, independant of the user's environment settings. It is sometimes neccesssary to override the locale when running build commands from make rules, but as maint.mk set LC_ALL, it is impossible to selectively overri