Re: [meta-oe][PATCH] openssl: fix multilib file install conflicts

[Peter Kjellerstedt <peter.kjellerstedt@axis.com>]

> -----Original Message-----
> From: openembedded-core-bounces@lists.openembedded.org <openembedded-
> core-bounces@lists.openembedded.org> On Behalf Of Randy MacLeod
> Sent: den 22 januari 2019 00:01
> To: Patches and discussions about the oe-core layer <openembedded-
> core@lists.openembedded.org>; Xulin Sun <xulin.sun@windriver.com>
> Subject: Re: [OE-core] [meta-oe][PATCH] openssl: fix multilib file
> install conflicts
> 
> Xulin,
> 
> Thanks for sending this patch.
> 
> On 1/17/19 9:22 PM, Xulin Sun wrote:
> > To avoid issue like below if run "bitbake lib32-wrlinux-image-glibc-
> std"
> 
> For future oe-core commits, please use a core-image-minimal
> example that will be more familiar to people outside of Wind River.
> 
> > with series userspace packages(LAMP,krb5...) added:
> >
> > Error: Transaction check error:
> >    file /usr/bin/c_rehash conflicts between attempted installs of
> >    lib32-openssl-bin-1.1.1-r0.armv7at2hf_neon and openssl-bin-1.1.1-
> r0.aarch64
> >
> 
> Also, while your commit log explains what you fixed, it
> doesn't describe the issue for those who don't know what c_rehash is so
> something like:
> 
>    Add multilib_script support for openssl's c_rehash which is
>    a perl script.
> 
> Followed by the example failure log that you showed.
> 
> 
> Since this hasn't been merged to master/master-next,
> it would be nice if you sent a v2.
> 
> Thanks,
> ../Randy
> 
> > Signed-off-by: Xulin Sun <xulin.sun@windriver.com>
> > ---
> >   meta/recipes-connectivity/openssl/openssl_1.1.1a.bb | 4 ++++
> >   1 file changed, 4 insertions(+)
> >
> > diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb
> b/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb
> > index 5c4e69cfb7..21359fa68a 100644
> > --- a/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb
> > +++ b/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb
> > @@ -206,3 +206,7 @@ RREPLACES_openssl-conf = "openssl10-conf"
> >   RCONFLICTS_openssl-conf = "openssl10-conf"
> >
> >   BBCLASSEXTEND = "native nativesdk"
> > +
> > +inherit multilib_script
> > +
> > +MULTILIB_SCRIPTS = "${PN}:${bindir}/c_rehash"

Shouldn't that be:

MULTILIB_SCRIPTS = "${PN}-bin:${bindir}/c_rehash"

given that openssl inherits lib_package and ${bindir}/c_rehash thus 
ends up in the ${PN}-bin package?

//Peter