Re: [OE-core] [PATCH v1] glibc-locale: work around host-user-contaminated QA failure
Hello, my bad, yes these changes are on some older branch and don't need upstream. Thanks, Hamza -Original Message- From: Richard Purdie Sent: Thursday, June 16, 2022 3:23 PM To: Hamza, Muhammad ; openembedded-core@lists.openembedded.org Cc: Larson, Chris Subject: Re: [OE-core] [PATCH v1] glibc-locale: work around host-user-contaminated QA failure On Thu, 2022-06-16 at 10:54 +0500, Muhammad Hamza wrote: > Work around long standing periodic host-user-contaminated QA failure > by explicitly correcting the ownership. > > See `glibc-locale: Rewrite do_install using install utility instead of > cp` on the oe-core mailing list for discussion. This should be dropped > when a real fix is implemented. > > Explicitly disable host-user-contaminated to further work around the > pseudo bug. With pseudo acting up, even if the ownership is correct, > it may well think it is not, so just sidestep the issue until upstream > fixes the root cause. > > Signed-off-by: Christopher Larson > > Signed-off-by: Muhammad Hamza > --- > meta/recipes-core/glibc/glibc-locale.inc | 17 + > 1 file changed, 17 insertions(+) > > diff --git a/meta/recipes-core/glibc/glibc-locale.inc > b/meta/recipes-core/glibc/glibc-locale.inc > index b8de7d3192..34c178268b 100644 > --- a/meta/recipes-core/glibc/glibc-locale.inc > +++ b/meta/recipes-core/glibc/glibc-locale.inc > @@ -69,6 +69,23 @@ FILES:localedef = "${bindir}/localedef" > > LOCALETREESRC = "${COMPONENTS_DIR}/${PACKAGE_ARCH}/glibc-stash-locale" > > +# Work around long standing periodic host-user-contaminated QA > +failure by # explicitly correcting the ownership. > +# > +# See `glibc-locale: Rewrite do_install using install utility instead > +of cp` # on the oe-core mailing list for discussion. This should be > +dropped when # a real fix is implemented. > + > +do_prep_locale_tree_append () { > + chown -R root:root $treedir > +} > + > +# Explicitly disable host-user-contaminated to further work around > +the # pseudo bug. With pseudo acting up, even if the ownership is > +correct, # it may well think it is not, so just sidestep the issue > +until upstream # fixes the root cause. > +ERROR_QA_remove = "host-user-contaminated" > + > copy_locale_files() { > local dir=$1 mode=$2 At a quick search I couldn't find the email you reference so it probably isn't recent. As Jacob mentions, this syntax is invalid for master. Was this not fixed properly by something like: https://git.yoctoproject.org/poky/commit/?id=2e75c084f2cfe7617de3b9a4b006caa3fee687a6 ? We're definitely not taking this without more details on what is going on and a correct patch as this looks like a workaround. Cheers, Richard -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#167005): https://lists.openembedded.org/g/openembedded-core/message/167005 Mute This Topic: https://lists.openembedded.org/mt/91793626/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH v1] glibc-locale: work around host-user-contaminated QA failure
On Thu, 2022-06-16 at 10:54 +0500, Muhammad Hamza wrote: > Work around long standing periodic host-user-contaminated QA failure by > explicitly correcting the ownership. > > See `glibc-locale: Rewrite do_install using install utility instead of cp` > on the oe-core mailing list for discussion. This should be dropped when > a real fix is implemented. > > Explicitly disable host-user-contaminated to further work around the > pseudo bug. With pseudo acting up, even if the ownership is correct, it > may well think it is not, so just sidestep the issue until upstream > fixes the root cause. > > Signed-off-by: Christopher Larson > > Signed-off-by: Muhammad Hamza > --- > meta/recipes-core/glibc/glibc-locale.inc | 17 + > 1 file changed, 17 insertions(+) > > diff --git a/meta/recipes-core/glibc/glibc-locale.inc > b/meta/recipes-core/glibc/glibc-locale.inc > index b8de7d3192..34c178268b 100644 > --- a/meta/recipes-core/glibc/glibc-locale.inc > +++ b/meta/recipes-core/glibc/glibc-locale.inc > @@ -69,6 +69,23 @@ FILES:localedef = "${bindir}/localedef" > > LOCALETREESRC = "${COMPONENTS_DIR}/${PACKAGE_ARCH}/glibc-stash-locale" > > +# Work around long standing periodic host-user-contaminated QA failure by > +# explicitly correcting the ownership. > +# > +# See `glibc-locale: Rewrite do_install using install utility instead of cp` > +# on the oe-core mailing list for discussion. This should be dropped when > +# a real fix is implemented. > + > +do_prep_locale_tree_append () { > + chown -R root:root $treedir > +} > + > +# Explicitly disable host-user-contaminated to further work around the > +# pseudo bug. With pseudo acting up, even if the ownership is correct, > +# it may well think it is not, so just sidestep the issue until upstream > +# fixes the root cause. > +ERROR_QA_remove = "host-user-contaminated" > + > copy_locale_files() { > local dir=$1 mode=$2 At a quick search I couldn't find the email you reference so it probably isn't recent. As Jacob mentions, this syntax is invalid for master. Was this not fixed properly by something like: https://git.yoctoproject.org/poky/commit/?id=2e75c084f2cfe7617de3b9a4b006caa3fee687a6 ? We're definitely not taking this without more details on what is going on and a correct patch as this looks like a workaround. Cheers, Richard -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#167004): https://lists.openembedded.org/g/openembedded-core/message/167004 Mute This Topic: https://lists.openembedded.org/mt/91793626/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH v1] glibc-locale: work around host-user-contaminated QA failure
On 6/16/22 07:54, Muhammad Hamza wrote: Work around long standing periodic host-user-contaminated QA failure by explicitly correcting the ownership. See `glibc-locale: Rewrite do_install using install utility instead of cp` on the oe-core mailing list for discussion. This should be dropped when a real fix is implemented. Explicitly disable host-user-contaminated to further work around the pseudo bug. With pseudo acting up, even if the ownership is correct, it may well think it is not, so just sidestep the issue until upstream fixes the root cause. Signed-off-by: Christopher Larson Signed-off-by: Muhammad Hamza --- meta/recipes-core/glibc/glibc-locale.inc | 17 + 1 file changed, 17 insertions(+) diff --git a/meta/recipes-core/glibc/glibc-locale.inc b/meta/recipes-core/glibc/glibc-locale.inc index b8de7d3192..34c178268b 100644 --- a/meta/recipes-core/glibc/glibc-locale.inc +++ b/meta/recipes-core/glibc/glibc-locale.inc @@ -69,6 +69,23 @@ FILES:localedef = "${bindir}/localedef" LOCALETREESRC = "${COMPONENTS_DIR}/${PACKAGE_ARCH}/glibc-stash-locale" +# Work around long standing periodic host-user-contaminated QA failure by +# explicitly correcting the ownership. +# +# See `glibc-locale: Rewrite do_install using install utility instead of cp` +# on the oe-core mailing list for discussion. This should be dropped when +# a real fix is implemented. + +do_prep_locale_tree_append () { + chown -R root:root $treedir +} + +# Explicitly disable host-user-contaminated to further work around the +# pseudo bug. With pseudo acting up, even if the ownership is correct, +# it may well think it is not, so just sidestep the issue until upstream +# fixes the root cause. +ERROR_QA_remove = "host-user-contaminated" + copy_locale_files() { local dir=$1 mode=$2 Given the append/remove override syntax used here, I guess this patch is aimed at an older Yocto release, and shouldn't be necessary for master ? Jacob -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#167001): https://lists.openembedded.org/g/openembedded-core/message/167001 Mute This Topic: https://lists.openembedded.org/mt/91793626/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-