diff mbox series

[meta-networking] net-snmp-native: dependency with libpci

Message ID 20250125115543.7605-1-gavrosc@yahoo.com
State New
Headers show
Series [meta-networking] net-snmp-native: dependency with libpci | expand

Commit Message

Christos Gavros Jan. 25, 2025, 11:55 a.m. UTC
Building of net-snmp-native aborted due to
missing dependency with libpci-native.
Fixed by changing the name to 'libpciaccess'
which is recipe's name

Signed-off-by: Christos Gavros <gavrosc@yahoo.com>
---
 meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Khem Raj Jan. 25, 2025, 5:07 p.m. UTC | #1
if its a build time issue then shouldn't it be added to DEPENDS ?

On Sat, Jan 25, 2025 at 3:56 AM Christos Gavros via
lists.openembedded.org <gavrosc=yahoo.com@lists.openembedded.org>
wrote:
>
> Building of net-snmp-native aborted due to
> missing dependency with libpci-native.
> Fixed by changing the name to 'libpciaccess'
> which is recipe's name
>
> Signed-off-by: Christos Gavros <gavrosc@yahoo.com>
> ---
>  meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
> index 06decbe852..cde1c1dfe8 100644
> --- a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
> +++ b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
> @@ -268,7 +268,7 @@ RDEPENDS:${PN}-server-snmpd += "net-snmp-mibs"
>  RDEPENDS:${PN}-server-snmptrapd += "net-snmp-server-snmpd ${PN}-lib-trapd"
>  RDEPENDS:${PN}-server += "net-snmp-server-snmpd net-snmp-server-snmptrapd"
>  RDEPENDS:${PN}-client += "net-snmp-mibs net-snmp-libs"
> -RDEPENDS:${PN}-libs += "libpci \
> +RDEPENDS:${PN}-libs += "libpciaccess \
>                          ${PN}-lib-netsnmp \
>                          ${PN}-lib-agent \
>                          ${PN}-lib-helpers \
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#115077): https://lists.openembedded.org/g/openembedded-devel/message/115077
> Mute This Topic: https://lists.openembedded.org/mt/110805858/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Khem Raj Jan. 25, 2025, 5:28 p.m. UTC | #2
On Sat, Jan 25, 2025 at 9:07 AM Khem Raj <raj.khem@gmail.com> wrote:
>
> if its a build time issue then shouldn't it be added to DEPENDS ?
>
> On Sat, Jan 25, 2025 at 3:56 AM Christos Gavros via
> lists.openembedded.org <gavrosc=yahoo.com@lists.openembedded.org>
> wrote:
> >
> > Building of net-snmp-native aborted due to
> > missing dependency with libpci-native.
> > Fixed by changing the name to 'libpciaccess'
> > which is recipe's name
> >
> > Signed-off-by: Christos Gavros <gavrosc@yahoo.com>
> > ---
> >  meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
> > index 06decbe852..cde1c1dfe8 100644
> > --- a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
> > +++ b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
> > @@ -268,7 +268,7 @@ RDEPENDS:${PN}-server-snmpd += "net-snmp-mibs"
> >  RDEPENDS:${PN}-server-snmptrapd += "net-snmp-server-snmpd ${PN}-lib-trapd"
> >  RDEPENDS:${PN}-server += "net-snmp-server-snmpd net-snmp-server-snmptrapd"
> >  RDEPENDS:${PN}-client += "net-snmp-mibs net-snmp-libs"
> > -RDEPENDS:${PN}-libs += "libpci \
> > +RDEPENDS:${PN}-libs += "libpciaccess \

looking further, libpci is a package built by pciutils recipe and its
added to DEPENDS here but only
for target recipe.  if we know this is not needed for native version
then we should mark the rdepends
for target too. So remove it from RDEPENDS:${PN}-libs  and Add
something like this

RDEPENDS:append:${PN}-libs:class-target = " libpci"

> >                          ${PN}-lib-netsnmp \
> >                          ${PN}-lib-agent \
> >                          ${PN}-lib-helpers \
> > --
> > 2.34.1
> >
> >
> > -=-=-=-=-=-=-=-=-=-=-=-
> > Links: You receive all messages sent to this group.
> > View/Reply Online (#115077): https://lists.openembedded.org/g/openembedded-devel/message/115077
> > Mute This Topic: https://lists.openembedded.org/mt/110805858/1997914
> > Group Owner: openembedded-devel+owner@lists.openembedded.org
> > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >
Christos Gavros Jan. 25, 2025, 9:09 p.m. UTC | #3
hi

I made the change you suggested and test it (bitbake net-snap-native) locally. The building is successful.
Give me some time to discuss it with 'CC' fellows for my understanding  and I will submit a v2 during the week.

Thank you
Christos
Christos Gavros Jan. 28, 2025, 9:09 p.m. UTC | #4
hi Raj,

i have made some further investigation with the support of Yoann Congal about this issue.
It seems that 'net-snmp-native' started breaking after the following patch which includes changes in native.bbclass to handle recipes/packages with suffix:
https://git.yoctoproject.org/poky/commit/?id=d26c72b7dccaf4802914dafcc95911f4d166086d

Before patch -> RDEPENDS: net-snmp-native-libs = libpci-native net-snmp-lib-netsnmp-native .....
After patch ->    RDEPENDS: net-snmp-libs-native = libpci-native net-snmp-lib-netsnmp-native .....

It looks that the issue was already there but after the patch it is exposed.

I have tested the changes you proposed locally and build successfully 'net-snmp' and 'net-snmp-native'.
I will prepare the v2 and send it unless you want me to test something more..

Br
Christos
diff mbox series

Patch

diff --git a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
index 06decbe852..cde1c1dfe8 100644
--- a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
+++ b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
@@ -268,7 +268,7 @@  RDEPENDS:${PN}-server-snmpd += "net-snmp-mibs"
 RDEPENDS:${PN}-server-snmptrapd += "net-snmp-server-snmpd ${PN}-lib-trapd"
 RDEPENDS:${PN}-server += "net-snmp-server-snmpd net-snmp-server-snmptrapd"
 RDEPENDS:${PN}-client += "net-snmp-mibs net-snmp-libs"
-RDEPENDS:${PN}-libs += "libpci \
+RDEPENDS:${PN}-libs += "libpciaccess \
                         ${PN}-lib-netsnmp \
                         ${PN}-lib-agent \
                         ${PN}-lib-helpers \