Hi guys,

As for now (master branch) 'PACKAGES' variable in net-snmp equals to:

```

$ bitbake -e net-snmp | grep "^PACKAGES="

PACKAGES="net-snmp-dbg net-snmp-staticdev net-snmp-dev net-snmp-doc 
net-snmp-locale  net-snmp net-snmp-libs net-snmp-mibs net-snmp-server 
net-snmp-client net-snmp-server-snmpd net-snmp-server-snmptrapd "
```


This seems to be wrong as many built packages from 'packages-split' directory 
of net-snmp are empty:

```
$ du -a --max-depth=1 . | sort -n
4    ./net-snmp
4    ./net-snmp-client.shlibdeps
4    ./net-snmp-doc
4    ./net-snmp-libs.shlibdeps
4    ./net-snmp-locale
4    ./net-snmp-server
4    ./net-snmp-server-snmpd.shlibdeps
4    ./net-snmp-server-snmptrapd.shlibdeps
4    ./net-snmp-staticdev
48    ./net-snmp-server-snmptrapd
72    ./net-snmp-server-snmpd
1208    ./net-snmp-dev
1480    ./net-snmp-client
1812    ./net-snmp-mibs
2872    ./net-snmp-libs
15308    ./net-snmp-dbg
```

Well, the culprit commit is 5eec0615e548f58ecdfadfc45af5805eeb58f69c where the 
below change has happened:
```
-PACKAGES = "${PN}-dbg ${PN}-doc ${PN}-dev ${PN}-staticdev ${PN}-static 
${PN}-libs \
-            ${PN}-mibs ${PN}-server ${PN}-client ${PN}-server-snmpd 
${PN}-server-snmptrapd"
+PACKAGES += "${PN}-libs ${PN}-mibs ${PN}-server ${PN}-client 
${PN}-server-snmpd ${PN}-server-snmptrapd"
```

This new 'PACKAGES' variable is wrong IMHO, because it contains
the 'net-snmp' package and other empty packages.


Please review (and apply if ok) the attached commit that fixes it.


Thanks,

--- Kosta Z.
From d5711e279d95307aa8bb2bbdd0bb311207287e62 Mon Sep 17 00:00:00 2001
From: Kosta Zertsekel <kzertse...@advaoptical.com>
Date: Sun, 18 Mar 2018 08:43:11 +0200
Subject: [PATCH] Fix PACKAGES for net-snmp to create only the correct net-snmp
 packages

As for now (master branch) 'PACKAGES' variable in net-snmp equals to:
```
$ bitbake -e net-snmp | grep "^PACKAGES="
PACKAGES="net-snmp-dbg net-snmp-staticdev net-snmp-dev net-snmp-doc \
          net-snmp-locale  net-snmp net-snmp-libs net-snmp-mibs     \
          net-snmp-server net-snmp-client net-snmp-server-snmpd     \
          net-snmp-server-snmptrapd"
```

This seems to be wrong as many built packages from
'packages-split' directory of net-snmp are empty:
```
$ du -a --max-depth=1 . | sort -n
4    ./net-snmp
4    ./net-snmp-client.shlibdeps
4    ./net-snmp-doc
4    ./net-snmp-libs.shlibdeps
4    ./net-snmp-locale
4    ./net-snmp-server
4    ./net-snmp-server-snmpd.shlibdeps
4    ./net-snmp-server-snmptrapd.shlibdeps
4    ./net-snmp-staticdev
48    ./net-snmp-server-snmptrapd
72    ./net-snmp-server-snmpd
1208    ./net-snmp-dev
1480    ./net-snmp-client
1812    ./net-snmp-mibs
2872    ./net-snmp-libs
15308    ./net-snmp-dbg
```

Well, the culprit commit is 5eec0615e548f58ecdfadfc45af5805eeb58f69c
where the below change has happened:
```
-PACKAGES = "${PN}-dbg ${PN}-doc ${PN}-dev ${PN}-staticdev ${PN}-static ${PN}-libs \
-            ${PN}-mibs ${PN}-server ${PN}-client ${PN}-server-snmpd ${PN}-server-snmptrapd"
+PACKAGES += "${PN}-libs ${PN}-mibs ${PN}-server ${PN}-client ${PN}-server-snmpd ${PN}-server-snmptrapd"
```

This new 'PACKAGES' variable is wrong IMHO, because it contains
the 'net-snmp' package and other empty packages.

Signed-off-by: Kosta Zertsekel <kzertse...@advaoptical.com>
---
 meta-networking/recipes-protocols/net-snmp/net-snmp_5.7.3.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.7.3.bb b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.7.3.bb
index 3c05874..bb113a8 100644
--- a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.7.3.bb
+++ b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.7.3.bb
@@ -165,7 +165,7 @@ net_snmp_sysroot_preprocess () {
     fi
 }
 
-PACKAGES += "${PN}-libs ${PN}-mibs ${PN}-server ${PN}-client ${PN}-server-snmpd ${PN}-server-snmptrapd"
+PACKAGES = "${PN}-dbg ${PN}-dev ${PN}-libs ${PN}-mibs ${PN}-client ${PN}-server-snmpd ${PN}-server-snmptrapd"
 
 # perl module
 PACKAGES += "${@bb.utils.contains('PACKAGECONFIG', 'perl', '${PN}-perl-modules', '', d)}"
-- 
2.7.4

-- 
_______________________________________________
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto

Reply via email to