Re: [PATCH v2] kbuild: replace LANG=C with LC_ALL=C

[Masahiro Yamada]

On Sat, May 22, 2021 at 3:34 AM
<patchwork-bot+linux-kselftest@xxxxxxxxxx> wrote:
>
> Hello:
>
> This patch was applied to shuah/linux-kselftest.git (refs/heads/next):
>
> On Fri, 30 Apr 2021 10:56:27 +0900 you wrote:
> > LANG gives a weak default to each LC_* in case it is not explicitly
> > defined. LC_ALL, if set, overrides all other LC_* variables.
> >
> >   LANG  <  LC_CTYPE, LC_COLLATE, LC_MONETARY, LC_NUMERIC, ...  <  LC_ALL
> >
> > This is why documentation such as [1] suggests to set LC_ALL in build
> > scripts to get the deterministic result.
> >
> > [...]
>
> Here is the summary with links:
>   - [v2] kbuild: replace LANG=C with LC_ALL=C
>     https://git.kernel.org/shuah/linux-kselftest/c/77a88274dc1a
>
> You are awesome, thank you!
> --


Huh?

This patch exists in Linus' tree.

Why is this going to the kselftest tree
in the first place?



commit 77a88274dc1a2cf3a775161d9a3242bc798ee680
Author: Masahiro Yamada <masahiroy@xxxxxxxxxx>
Date:   Fri Apr 30 10:56:27 2021 +0900

    kbuild: replace LANG=C with LC_ALL=C

    LANG gives a weak default to each LC_* in case it is not explicitly
    defined. LC_ALL, if set, overrides all other LC_* variables.

      LANG  <  LC_CTYPE, LC_COLLATE, LC_MONETARY, LC_NUMERIC, ...  <  LC_ALL

    This is why documentation such as [1] suggests to set LC_ALL in build
    scripts to get the deterministic result.

    LANG=C is not strong enough to override LC_* that may be set by end
    users.

    [1]: https://reproducible-builds.org/docs/locales/

    Signed-off-by: Masahiro Yamada <masahiroy@xxxxxxxxxx>
    Acked-by: Michael Ellerman <mpe@xxxxxxxxxxxxxx> (powerpc)
    Reviewed-by: Matthias Maennich <maennich@xxxxxxxxxx>
    Acked-by: Matthieu Baerts <matthieu.baerts@xxxxxxxxxxxx> (mptcp)
    Reviewed-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>






--
Best Regards
Masahiro Yamada