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
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 \