[meta-networking][PATCH 1/1] net-snmp: fix the replacement in net-snmp-config

[meta-networking][PATCH 1/1] net-snmp: fix the replacement in net-snmp-config

[rongqing.li]

From: Roy Li <rongqing.li@windriver.com>

when net-snmp-config is used to configure by other package, and fail since
/libnl3 is not found, in fact, it should be -I/usr/include/libnl3, and is
modified as /libnl3 incorrectly.

fix this replacement by adding = after -I to consider the sysroot

Signed-off-by: Roy Li <rongqing.li@windriver.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 b32d842..ebd0c68 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
@@ -63,7 +63,7 @@ do_install_append() {
     install -m 755 ${WORKDIR}/init ${D}${sysconfdir}/init.d/snmpd
     install -m 644 ${WORKDIR}/snmpd.conf ${D}${sysconfdir}/snmp/
     install -m 644 ${WORKDIR}/snmptrapd.conf ${D}${sysconfdir}/snmp/
-    sed -e "s@-I/usr/include@@g" \
+    sed -e "s@-I/usr/include@-I=/usr/include@g" \
         -e "s@^prefix=.*@prefix=${STAGING_DIR_HOST}@g" \
         -e "s@^exec_prefix=.*@exec_prefix=${STAGING_DIR_HOST}@g" \
         -e "s@^includedir=.*@includedir=${STAGING_INCDIR}@g" \
-- 
1.9.1

Re: [meta-networking][PATCH 1/1] net-snmp: fix the replacement in net-snmp-config

[Khem Raj]

On Wed, Dec 30, 2015 at 5:26 PM,  <rongqing.li@windriver.com> wrote:
> -    sed -e "s@-I/usr/include@@g" \
> +    sed -e "s@-I/usr/include@-I=/usr/include@g" \

Can you rather replace it with ${STAGING_INCDIR} -I= is too gcc
specific, wont work with Clang e.g.

Re: [meta-networking][PATCH 1/1] net-snmp: fix the replacement in net-snmp-config

[Rongqing Li]

On 2015年12月31日 14:17, Khem Raj wrote:
> On Wed, Dec 30, 2015 at 5:26 PM,  <rongqing.li@windriver.com> wrote:
>> -    sed -e "s@-I/usr/include@@g" \
>> +    sed -e "s@-I/usr/include@-I=/usr/include@g" \
>
> Can you rather replace it with ${STAGING_INCDIR} -I= is too gcc
> specific, wont work with Clang e.g.
>

Ok

thanks

-- 
Best Reagrds,
Roy | RongQing Li