Re: [PATCH] build.sh: Fix perl complaints about locale

[Alexander Smirnov <asmirnov@ilbers.de>]

On 11/13/2017 04:44 PM, Alexander Smirnov wrote:
> Hi,
> 
> On 11/13/2017 04:19 PM, Christian Storm wrote:
>>> On 11/13/2017 03:21 PM, Christian Storm wrote:
>>>> The log.do_build is flooded by perl's locale complaints:
>>>>
>>>>       perl: warning: Setting locale failed.
>>>>       perl: warning: Please check that your locale settings:
>>>>               LANGUAGE = (unset),
>>>>               LC_ALL = "en_US.UTF-8",
>>>>               LANG = (unset)
>>>>           are supported and installed on your system.
>>>>       perl: warning: Falling back to the standard locale ("C").
>>>>
>>>> Make perl happy by explicitly giving it the C locale.
>>>>
>>>> Signed-off-by: Christian Storm <christian.storm@siemens.com>
>>>> ---
>>>>    meta/recipes-devtools/buildchroot/files/build.sh | 2 +-
>>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/meta/recipes-devtools/buildchroot/files/build.sh 
>>>> b/meta/recipes-devtools/buildchroot/files/build.sh
>>>> index 19d554e..e53d359 100644
>>>> --- a/meta/recipes-devtools/buildchroot/files/build.sh
>>>> +++ b/meta/recipes-devtools/buildchroot/files/build.sh
>>>> @@ -23,4 +23,4 @@ for i in configure aclocal.m4 Makefile.am 
>>>> Makefile.in; do
>>>>    done
>>>>    # Build the package
>>>> -dpkg-buildpackage
>>>> +LC_ALL=C LANGUAGE=C LANG=C dpkg-buildpackage
>>>>
>>>
>>> There is code that should setup locales for current buildchroot:
>>>
>>> https://github.com/ilbers/isar/blob/master/meta/recipes-devtools/buildchroot/files/configscript.sh#L8 
>>>
>>>
>>> So it would be nice to fix this globally instead of local solution.
>>
>> Sure, but this alone doesn't solve the problem as you have to put the
>> locale into every (sudo) environment. Then, what about
> 
> Hmm, it works as it is for archs different from host machine. I've 
> attached 3 build logs for hello application in Stretch: amd64, arm and 
> i386 and only *amd64* has locale flood. The others are OK. So that's why 
> I think it's rather bug in configscript.

Working on PRoot I've found the following multistrap option:

"ignorenativearch=true"

Probably it could somehow help here...

Alex

>>
>> --- a/meta/recipes-devtools/buildchroot/files/build.sh
>> +++ b/meta/recipes-devtools/buildchroot/files/build.sh
>> @@ -3,6 +3,8 @@
>>   # This software is a part of ISAR.
>>   # Copyright (C) 2015-2017 ilbers GmbH
>> +[ -r /etc/default/locale ] && . /etc/default/locale
>> +
>>   # Go to build directory
>>   cd $1
>> -- 
>>
>> for which I could send a patch if this is OK for you..
>>
>>
>>> BTW, which target distro do you use, Stretch? I already have this
>>> issue in todo list, but you are welcome to fix this. :-)
>>
>> Yes, stretch.
>>
>>
>>
>> Besten Gruß,
>>     Christian
>>
>