openembedded-core.lists.openembedded.org archive mirror
 help / color / mirror / Atom feed
* [oe-core][PATCH] bzip2: add libbz2.so.1.0 link for compat
@ 2024-03-16  0:19 rs
  2024-03-16 16:52 ` Alexander Kanavin
  2024-03-17  9:28 ` Richard Purdie
  0 siblings, 2 replies; 6+ messages in thread
From: rs @ 2024-03-16  0:19 UTC (permalink / raw)
  To: richard.purdie, reatmon, denys; +Cc: openembedded-core

From: Randolph Sapp <rs@ti.com>

Add a link from libbz2.so.1.0 to libbz2.so.${PV} for compatibility with
other distributions.

A handfull of distributions are still using the "Makefile-libbz2_so" to
generate libbz2 and bzip2-shared. This creates a library with a soname
of "libbz2.so.1.0". This library link is not provide by default in
autotools builds so add one here as Gentoo does now [1].

This has been addressed upstream already, but there hasn't been a stable
release since then [2].

[1] https://bugs.gentoo.org/338321
[2] https://gitlab.com/bzip2/bzip2/-/merge_requests/42

Signed-off-by: Randolph Sapp <rs@ti.com>
---
 meta/recipes-extended/bzip2/bzip2_1.0.8.bb | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
index 4e3a06f240..4453171701 100644
--- a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
+++ b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
@@ -62,6 +62,11 @@ do_install_ptest () {
 	sed -i -e "s|^Makefile:|_Makefile:|" ${D}${PTEST_PATH}/Makefile
 }
 
+# add the following link for cross-distro compatibility
+do_install:append () {
+	ln -s libbz2.so.${PV} "${D}${libdir}/libbz2.so.1.0"
+}
+
 FILES:libbz2 = "${libdir}/lib*${SOLIBS}"
 
 RDEPENDS:${PN}-ptest += "make bash"
-- 
2.44.0



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [oe-core][PATCH] bzip2: add libbz2.so.1.0 link for compat
  2024-03-16  0:19 [oe-core][PATCH] bzip2: add libbz2.so.1.0 link for compat rs
@ 2024-03-16 16:52 ` Alexander Kanavin
  2024-03-18 19:15   ` Randolph Sapp
  2024-03-17  9:28 ` Richard Purdie
  1 sibling, 1 reply; 6+ messages in thread
From: Alexander Kanavin @ 2024-03-16 16:52 UTC (permalink / raw)
  To: rs; +Cc: richard.purdie, reatmon, denys, openembedded-core

It's better to backport the upstream patch, as nobody's going to
remember why do_install snippet is there, or to remove it when there's
a new release (if it doesn't cause installation conflicts).

Alex

On Sat, 16 Mar 2024 at 01:19, Randolph Sapp via lists.openembedded.org
<rs=ti.com@lists.openembedded.org> wrote:
>
> From: Randolph Sapp <rs@ti.com>
>
> Add a link from libbz2.so.1.0 to libbz2.so.${PV} for compatibility with
> other distributions.
>
> A handfull of distributions are still using the "Makefile-libbz2_so" to
> generate libbz2 and bzip2-shared. This creates a library with a soname
> of "libbz2.so.1.0". This library link is not provide by default in
> autotools builds so add one here as Gentoo does now [1].
>
> This has been addressed upstream already, but there hasn't been a stable
> release since then [2].
>
> [1] https://bugs.gentoo.org/338321
> [2] https://gitlab.com/bzip2/bzip2/-/merge_requests/42
>
> Signed-off-by: Randolph Sapp <rs@ti.com>
> ---
>  meta/recipes-extended/bzip2/bzip2_1.0.8.bb | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
> index 4e3a06f240..4453171701 100644
> --- a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
> +++ b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
> @@ -62,6 +62,11 @@ do_install_ptest () {
>         sed -i -e "s|^Makefile:|_Makefile:|" ${D}${PTEST_PATH}/Makefile
>  }
>
> +# add the following link for cross-distro compatibility
> +do_install:append () {
> +       ln -s libbz2.so.${PV} "${D}${libdir}/libbz2.so.1.0"
> +}
> +
>  FILES:libbz2 = "${libdir}/lib*${SOLIBS}"
>
>  RDEPENDS:${PN}-ptest += "make bash"
> --
> 2.44.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#197252): https://lists.openembedded.org/g/openembedded-core/message/197252
> Mute This Topic: https://lists.openembedded.org/mt/104959821/1686489
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [oe-core][PATCH] bzip2: add libbz2.so.1.0 link for compat
  2024-03-16  0:19 [oe-core][PATCH] bzip2: add libbz2.so.1.0 link for compat rs
  2024-03-16 16:52 ` Alexander Kanavin
@ 2024-03-17  9:28 ` Richard Purdie
  2024-03-18 19:13   ` Randolph Sapp
  1 sibling, 1 reply; 6+ messages in thread
From: Richard Purdie @ 2024-03-17  9:28 UTC (permalink / raw)
  To: rs, reatmon, denys; +Cc: openembedded-core

On Fri, 2024-03-15 at 19:19 -0500, rs@ti.com wrote:
> From: Randolph Sapp <rs@ti.com>
> 
> Add a link from libbz2.so.1.0 to libbz2.so.${PV} for compatibility
> with
> other distributions.
> 
> A handfull of distributions are still using the "Makefile-libbz2_so" to
> generate libbz2 and bzip2-shared. This creates a library with a soname
> of "libbz2.so.1.0". This library link is not provide by default in
> autotools builds so add one here as Gentoo does now [1].
> 
> This has been addressed upstream already, but there hasn't been a stable
> release since then [2].
> 
> [1] https://bugs.gentoo.org/338321
> [2] https://gitlab.com/bzip2/bzip2/-/merge_requests/42
> 
> Signed-off-by: Randolph Sapp <rs@ti.com>
> ---
>  meta/recipes-extended/bzip2/bzip2_1.0.8.bb | 5 +++++
>  1 file changed, 5 insertions(+)

I guess by "cross distro", you mean that you have some binaries which
need to run on multiple distros, some of which have this naming and
some of which do not?

I am a bit worried about this as it is the kind of thing we add, then
totally forget about removing so it gets carried even after it no
longer makes sense (and the library name/version changes again for
example).

Ugly as it may be, I'm wondering if there should be a separate recipe
to handle this link which the binaries could depend upon. Even a
separate package for it and an error if the target lib doesn't exist
might be better.

Cheers,

Richard




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [oe-core][PATCH] bzip2: add libbz2.so.1.0 link for compat
  2024-03-17  9:28 ` Richard Purdie
@ 2024-03-18 19:13   ` Randolph Sapp
  0 siblings, 0 replies; 6+ messages in thread
From: Randolph Sapp @ 2024-03-18 19:13 UTC (permalink / raw)
  To: Richard Purdie, reatmon, denys; +Cc: openembedded-core

On Sun Mar 17, 2024 at 4:28 AM CDT, Richard Purdie wrote:
> On Fri, 2024-03-15 at 19:19 -0500, rs@ti.com wrote:
> > From: Randolph Sapp <rs@ti.com>
> > 
> > Add a link from libbz2.so.1.0 to libbz2.so.${PV} for compatibility
> > with
> > other distributions.
> > 
> > A handfull of distributions are still using the "Makefile-libbz2_so" to
> > generate libbz2 and bzip2-shared. This creates a library with a soname
> > of "libbz2.so.1.0". This library link is not provide by default in
> > autotools builds so add one here as Gentoo does now [1].
> > 
> > This has been addressed upstream already, but there hasn't been a stable
> > release since then [2].
> > 
> > [1] https://bugs.gentoo.org/338321
> > [2] https://gitlab.com/bzip2/bzip2/-/merge_requests/42
> > 
> > Signed-off-by: Randolph Sapp <rs@ti.com>
> > ---
> >  meta/recipes-extended/bzip2/bzip2_1.0.8.bb | 5 +++++
> >  1 file changed, 5 insertions(+)
>
> I guess by "cross distro", you mean that you have some binaries which
> need to run on multiple distros, some of which have this naming and
> some of which do not?
>
> I am a bit worried about this as it is the kind of thing we add, then
> totally forget about removing so it gets carried even after it no
> longer makes sense (and the library name/version changes again for
> example).
>
> Ugly as it may be, I'm wondering if there should be a separate recipe
> to handle this link which the binaries could depend upon. Even a
> separate package for it and an error if the target lib doesn't exist
> might be better.
>
> Cheers,
>
> Richard

Yeah, anything compiled on Debian/Arch will run into issues looking up that so
name when running under Yocto.

Oddly enough though, both Fedora and Gentoo seem to be using the new soname. I'm
not entirely sure if this is actually that common now. Maybe this should be
ignored.


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [oe-core][PATCH] bzip2: add libbz2.so.1.0 link for compat
  2024-03-16 16:52 ` Alexander Kanavin
@ 2024-03-18 19:15   ` Randolph Sapp
  0 siblings, 0 replies; 6+ messages in thread
From: Randolph Sapp @ 2024-03-18 19:15 UTC (permalink / raw)
  To: Alexander Kanavin; +Cc: richard.purdie, reatmon, denys, openembedded-core

On Sat Mar 16, 2024 at 11:52 AM CDT, Alexander Kanavin wrote:
> It's better to backport the upstream patch, as nobody's going to
> remember why do_install snippet is there, or to remove it when there's
> a new release (if it doesn't cause installation conflicts).
>
> Alex

I would backport the upstream patch, if it wasn't dependant on a migration to
Cmake. I feel like that's a bit much.

Randolph

> On Sat, 16 Mar 2024 at 01:19, Randolph Sapp via lists.openembedded.org
> <rs=ti.com@lists.openembedded.org> wrote:
> >
> > From: Randolph Sapp <rs@ti.com>
> >
> > Add a link from libbz2.so.1.0 to libbz2.so.${PV} for compatibility with
> > other distributions.
> >
> > A handfull of distributions are still using the "Makefile-libbz2_so" to
> > generate libbz2 and bzip2-shared. This creates a library with a soname
> > of "libbz2.so.1.0". This library link is not provide by default in
> > autotools builds so add one here as Gentoo does now [1].
> >
> > This has been addressed upstream already, but there hasn't been a stable
> > release since then [2].
> >
> > [1] https://bugs.gentoo.org/338321
> > [2] https://gitlab.com/bzip2/bzip2/-/merge_requests/42
> >
> > Signed-off-by: Randolph Sapp <rs@ti.com>
> > ---
> >  meta/recipes-extended/bzip2/bzip2_1.0.8.bb | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
> > index 4e3a06f240..4453171701 100644
> > --- a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
> > +++ b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
> > @@ -62,6 +62,11 @@ do_install_ptest () {
> >         sed -i -e "s|^Makefile:|_Makefile:|" ${D}${PTEST_PATH}/Makefile
> >  }
> >
> > +# add the following link for cross-distro compatibility
> > +do_install:append () {
> > +       ln -s libbz2.so.${PV} "${D}${libdir}/libbz2.so.1.0"
> > +}
> > +
> >  FILES:libbz2 = "${libdir}/lib*${SOLIBS}"
> >
> >  RDEPENDS:${PN}-ptest += "make bash"
> > --
> > 2.44.0
> >
> >
> > -=-=-=-=-=-=-=-=-=-=-=-
> > Links: You receive all messages sent to this group.
> > View/Reply Online (#197252): https://lists.openembedded.org/g/openembedded-core/message/197252
> > Mute This Topic: https://lists.openembedded.org/mt/104959821/1686489
> > Group Owner: openembedded-core+owner@lists.openembedded.org
> > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [OE-core] [PATCH] bzip2: add libbz2.so.1.0 link for compat
       [not found] ` <182C83EDE0EF5771.27055@lists.openembedded.org>
@ 2025-03-14  0:40   ` Randolph Sapp
  0 siblings, 0 replies; 6+ messages in thread
From: Randolph Sapp @ 2025-03-14  0:40 UTC (permalink / raw)
  To: rs, otavio.salvador, alex.kanavin
  Cc: richard.purdie, ross.burton, alex, otavio, kexin.hao, afd,
	detheridge, denis, reatmon, openembedded-core, vijayp

On Thu Mar 13, 2025 at 7:29 PM CDT, Randolph Sapp via lists.openembedded.org wrote:
> From: Randolph Sapp <rs@ti.com>
>
> Add a link from libbz2.so.1.0 to libbz2.so.${PV} for compatibility with
> other distributions.
>
> A handfull of distributions are still using the "Makefile-libbz2_so" to
> generate libbz2 and bzip2-shared. This creates a library with a soname
> of "libbz2.so.1.0". This library link is not provide by default in
> autotools builds so add one here as Gentoo does now [1].
>
> This has been addressed upstream already, but there hasn't been a stable
> release since then [2].
>
> [1] https://bugs.gentoo.org/338321
> [2] https://gitlab.com/bzip2/bzip2/-/merge_requests/42
>
> Signed-off-by: Randolph Sapp <rs@ti.com>
> ---
>  meta/recipes-extended/bzip2/bzip2_1.0.8.bb | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
> index 78138d1543..02d81e7069 100644
> --- a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
> +++ b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb
> @@ -62,6 +62,11 @@ do_install_ptest () {
>  	sed -i -e "s|^Makefile:|_Makefile:|" ${D}${PTEST_PATH}/Makefile
>  }
>  
> +# add the following link for cross-distro compatibility
> +do_install:append () {
> +	ln -s libbz2.so.${PV} "${D}${libdir}/libbz2.so.1.0"
> +}
> +
>  FILES:libbz2 = "${libdir}/lib*${SOLIBS}"
>  
>  RDEPENDS:${PN}-ptest += "make bash"

Sorry for the spam. These are not part of the RFC series and should be ignored.
Old patches I forgot to clean out of my pwd.


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2025-03-14  0:40 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-16  0:19 [oe-core][PATCH] bzip2: add libbz2.so.1.0 link for compat rs
2024-03-16 16:52 ` Alexander Kanavin
2024-03-18 19:15   ` Randolph Sapp
2024-03-17  9:28 ` Richard Purdie
2024-03-18 19:13   ` Randolph Sapp
  -- strict thread matches above, loose matches on Subject: below --
2025-03-14  0:29 [oe-core][RFC 0/3] Display manager proposal for x11 and wayland rs
     [not found] ` <182C83EDE0EF5771.27055@lists.openembedded.org>
2025-03-14  0:40   ` [OE-core] [PATCH] bzip2: add libbz2.so.1.0 link for compat Randolph Sapp

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).