Re: [PATCH] glibc-package.inc: correct intention for deleting /usr/lib as needed

["Belal, Awais" <Awais_Belal@mentor.com>]

Hi Ross,

When do you think this would go in?

BR,
Awais

________________________________________
From: openembedded-core-bounces@lists.openembedded.org <openembedded-core-bounces@lists.openembedded.org> on behalf of Belal, Awais
Sent: Friday, September 21, 2018 6:31 PM
To: Burton, Ross
Cc: OE-core
Subject: Re: [OE-core] [PATCH] glibc-package.inc: correct intention for deleting /usr/lib as needed

>> That's better than HTML!
Attached.

BR,
Awais

________________________________________
From: Burton, Ross <ross.burton@intel.com>
Sent: Thursday, September 20, 2018 7:46 PM
To: Belal, Awais
Cc: OE-core
Subject: Re: [OE-core] [PATCH] glibc-package.inc: correct intention for deleting /usr/lib as needed

That's better than HTML!
On Thu, 20 Sep 2018 at 15:04, Belal, Awais <Awais_Belal@mentor.com> wrote:
>
> Hi Ross,
>
> >> Can you resend using git-send-email, as this patch is encoded in HTML.
> My SMTP server is acting up these days, should I simply resend the patch here as an attachment?
>
> BR,
> Awais
>
> ________________________________________
> From: Burton, Ross [ross.burton@intel.com]
> Sent: Thursday, September 20, 2018 4:10 PM
> To: Belal, Awais
> Cc: OE-core
> Subject: Re: [OE-core] [PATCH] glibc-package.inc: correct intention for deleting /usr/lib as needed
>
> Can you resend using git-send-email, as this patch is encoded in HTML.
>
> Ross
> On Tue, 18 Sep 2018 at 11:53, Belal, Awais <Awais_Belal@mentor.com> wrote:
> >
> > In case the baselib is lib64 we would want to delete /usr/lib
> > after removing the /usr/lib/locale dir and the implementation
> > wanted to do that earlier as well but the fault was checking
> > an already removed dir (/usr/lib/locale) before trying to
> > remove /usr/lib as that check would always fail.
> > Now we simply try to delete /usr/lib after deleting
> > /usr/lib/locale to make sure it deletes cleanly and is empty
> > at the time of deletion.
> >
> > Signed-off-by: Awais Belal <awais_belal@mentor.com>
> > ---
> >  meta/recipes-core/glibc/glibc-package.inc | 9 ++++-----
> >  1 file changed, 4 insertions(+), 5 deletions(-)
> >
> > diff --git a/meta/recipes-core/glibc/glibc-package.inc b/meta/recipes-core/glibc/glibc-package.inc
> > index 9ea41b7..22a59d2 100644
> > --- a/meta/recipes-core/glibc/glibc-package.inc
> > +++ b/meta/recipes-core/glibc/glibc-package.inc
> > @@ -207,11 +207,10 @@ do_poststash_install_cleanup () {
> >      rm -rf ${D}/${localedir}
> >      rm -rf ${D}${datadir}/locale
> >      if [ "${libdir}" != "${exec_prefix}/lib" ]; then
> > -        if [ -d ${D}${exec_prefix}/lib/locale ] ; then
> > -            rm -rf ${D}${exec_prefix}/lib/locale
> > -            # error out if directory isn't empty
> > -            rm -f ${D}${exec_prefix}/lib
> > -        fi
> > +        # error out if directory isn't empty
> > +        # this dir should only contain locale dir
> > +        # which has been deleted in the previous step
> > +        rmdir ${D}${exec_prefix}/lib
> >      fi
> >  }
> >  addtask do_poststash_install_cleanup after do_stash_locale do_install before do_populate_sysroot do_package
> > --
> > 2.7.4
> >
> >
> > --
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core