]> code.ossystems Code Review - openembedded-core.git/commitdiff
glibc-package: Avoid race sstate races with do_stash_locale
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Wed, 8 Feb 2017 12:29:04 +0000 (12:29 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 5 Apr 2018 14:11:16 +0000 (15:11 +0100)
The change to make do_stash_locale an sstate task between do_install and
do_package has some unforeseen problems since the function deletes part of
${D} but may or may not run depending on whether the task is installed from
sstate.

This cleans up the current "pre packaging" function to be more deterministic
and result in the same set of files, whichever code path we end up reaching
that point by. Its not an ideal sitation but it should avoid the race
failures we've seen on some builds.

(From OE-Core rev: 2a60bba00da9c4e2b1a1ce30e509dba82cc6c94a)

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Armin Kuster <akuster808@gmail.com>
meta/recipes-core/glibc/glibc-package.inc

index b8239e0f025af6e248093b5a866d416160160992..6f4e71d1def758be5114702ac3c97c6977f535cf 100644 (file)
@@ -192,18 +192,23 @@ python do_stash_locale_setscene () {
 }
 addtask do_stash_locale_setscene
 
-PACKAGE_PREPROCESS_FUNCS += "glibc_package_preprocess"
-
-glibc_package_preprocess () {
+do_poststash_install_cleanup () {
+       # Remove all files which do_stash_locale would remove (mv)
+       # since that task could have come from sstate and not get run.
        for i in ${bashscripts}; do
-           rm -f ${PKGD}${bindir}/$i
+           rm -f ${D}${bindir}/$i
        done
-       rm -rf ${PKGD}/${localedir}
+       rm -f ${D}${bindir}/localedef
+       rm -rf ${D}${datadir}/i18n
+       rm -rf ${D}${libdir}/gconv
+       rm -rf ${D}/${localedir}
+       rm -rf ${D}${datadir}/locale
        if [ "${libdir}" != "${exec_prefix}/lib" ]; then
                # This dir only exists to hold locales
-               rm -rf ${PKGD}${exec_prefix}/lib
+               rm -rf ${D}${exec_prefix}/lib
        fi
 }
+addtask do_poststash_install_cleanup after do_stash_locale do_install before do_populate_sysroot do_package
 
 pkg_postinst_nscd () {
        if [ -z "$D" ]; then