I *think* it's sufficient as-is.

LC_ALL (and several other variables) are set via
libltdl/config/ltmain.m4sh circa line 114

looks like configure (which is ultimately where libtool.m4 gets
included) does it too, via autoconf _AS_SHELL_SANITIZE... I can't find
the exact path that causes it to be included, but setting LC_ALL is at
the start of autoconf script I can find.


(And oh, my build scripts -- independent of all of the above -- do it,
too, so *I'm* covered N different ways...  8-)



chris

On Mon, Mar 15, 2010 at 18:12, Eric Blake <ebl...@redhat.com> wrote:
> On 03/15/2010 01:12 PM, Chris Demetriou wrote:
>> The attached patch sorts the output of 'find' in all cases (where it's
>> not already sorted), so as to produce deterministic results.
>
> Does libtool globally force the C locale, or do you need to use:
>
> LC_ALL=C sort
>
> throughout your patch to further ensure deterministic behavior across
> different default locales?
>
> --
> Eric Blake   ebl...@redhat.com    +1-801-349-2682
> Libvirt virtualization library http://libvirt.org
>
>


Reply via email to