]> code.ossystems Code Review - openembedded-core.git/commit
glibc / glibc-locale: Fix stash_locale determinism problems
authorJason Wessel <jason.wessel@windriver.com>
Mon, 8 Jul 2019 19:16:12 +0000 (12:16 -0700)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Wed, 10 Jul 2019 08:55:45 +0000 (09:55 +0100)
commitfedc57a41a15bca1d96d14e25e2df0bb1eca904d
tree28249a23f1f310130a56bf6bf37f7e099eafa5a0
parent76350b19945db458c60d3f233770543c00cb2067
glibc / glibc-locale: Fix stash_locale determinism problems

When using sstate, or performing an incremental build any change to
the do_stash_locale() will cause a build failure because
do_stash_locale() was destroying the results obtained from the
do_install() with several mv operations.  A recent change to
do_stash_locale() for a different problem illustrated a number of
build failures for users in the community.

To fix the problem, do_stash_locale() must use copy operations instead
of the mv operations.  Because this is changed to a copy, the sysroot
and package stage need to remove the files that would have been
previously removed.  The correct "fixup" code to deal with the removal
already existed in the previous do_poststash_install_cleanup().  All
that needed change was the path to where to remove the files
from the sysroot and package stages.

In order to force a re-compilation of glibc some unused white space
was removed from do_compile() for glibc.  I could not find any other
way around this and we don't want to have all the community folks to
have another iteration where they have to remove their tmp directories
or purge some portion of the sstate.  It also makes this change
bisectable. If the change to the glibc is not included, it will fail
with the following message:

=====
| DEBUG: Executing shell function do_prep_locale_tree
| tar: i18n: Cannot stat: No such file or directory
| tar: Exiting with failure status due to previous errors
| gzip: /poky/build/tmp/work/core2-64-poky-linux/glibc-locale/2.29-r0/locale-tree//usr/share/i18n/charmaps/*gz.gz: No such file or directory
=====

After this one time change I tested changing only the
do_stash_locale() function and it now works well because it is
deterministically operating off the sstate data or a local build.

Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/recipes-core/glibc/glibc-package.inc
meta/recipes-core/glibc/glibc_2.29.bb