]> 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)
committerArmin Kuster <akuster808@gmail.com>
Wed, 4 Sep 2019 15:22:03 +0000 (08:22 -0700)
commit7e62ca2a518f1fe6e2f3c7098d17ed0b73ece48c
tree24ed3376e8f591928689d8a084b8df2db6cec533
parent36c223ecc5349478821a7a64a4ff14bbeae0a7da
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>
Signed-off-by: Armin Kuster <akuster808@gmail.com>
meta/recipes-core/glibc/glibc-package.inc
meta/recipes-core/glibc/glibc_2.29.bb