Re: [oe] [PATCH] netcat-openbsd: fix implicit dependency on pkg-config

2017-02-18 Thread Khem Raj
On Sat, Feb 18, 2017 at 5:12 PM Paul Gortmaker 
wrote:

> In the report at http://errors.yoctoproject.org/Errors/Details/130673/
> we see the following:
>
> i586-oe-linux-gcc  -m32 -march=i586  netcat.o atomicio.o socks.o
> `pkg-config --libs libbsd` -lresolv -o nc
> /bin/sh: 1: pkg-config: not found
>
> What follows is a bunch of link errors for BSD library functions, as
> the backtick didn't return anything but an empty string and hence no
> BSD library was involved in the link.
>
> I was able to reproduce this by temporarily removing my host version
> of pkg-config, and then validated that this fix solves it while still
> having the host binary removed.  So it should fix the reported issue.
> I also confirmed pkg-config was in the sysroot after the change.
>
> Cc: Martin Jansa 
> Signed-off-by: Paul Gortmaker 
>
> diff --git a/meta-networking/recipes-support/netcat/
> netcat-openbsd_1.105.bb b/meta-networking/recipes-support/netcat/
> netcat-openbsd_1.105.bb
> index bb21e0cfeede..06206a5a59d1 100644
> --- a/meta-networking/recipes-support/netcat/netcat-openbsd_1.105.bb
> +++ b/meta-networking/recipes-support/netcat/netcat-openbsd_1.105.bb
> @@ -4,7 +4,7 @@ HOMEPAGE = "http://ftp.debian.org";
>  LICENSE = "BSD-3-Clause"
>  LIC_FILES_CHKSUM =
> "file://../debian/copyright;md5=ee6bbaacb5db5f2973818f0902c3ae6f"
>
> -DEPENDS += "glib-2.0 libbsd"
> +DEPENDS += "glib-2.0 libbsd pkgconfig-native"


I think inherit pkgconfig should be the way to go here

>
>
>  SRC_URI =
> "${DEBIAN_MIRROR}/main/n/netcat-openbsd/netcat-openbsd_${PV}.orig.tar.gz;name=netcat
> \
>
> ${DEBIAN_MIRROR}/main/n/netcat-openbsd/netcat-openbsd_${PV}-7.debian.tar.gz;name=netcat-patch"
> @@ -42,5 +42,3 @@ do_install() {
>  install -m 755 ${S}/nc ${D}${bindir}/nc.${BPN}
>  }
>  ALTERNATIVE_PRIORITY = "60"
> -
> -PNBLACKLIST[netcat-openbsd] ?= "Fails to build with RSS
> http://errors.yoctoproject.org/Errors/Details/130673/";
> --
> 2.7.4
>
> --
> ___
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [PATCH] netcat-openbsd: fix implicit dependency on pkg-config

2017-02-18 Thread Paul Gortmaker
In the report at http://errors.yoctoproject.org/Errors/Details/130673/
we see the following:

i586-oe-linux-gcc  -m32 -march=i586  netcat.o atomicio.o socks.o 
`pkg-config --libs libbsd` -lresolv -o nc
/bin/sh: 1: pkg-config: not found

What follows is a bunch of link errors for BSD library functions, as
the backtick didn't return anything but an empty string and hence no
BSD library was involved in the link.

I was able to reproduce this by temporarily removing my host version
of pkg-config, and then validated that this fix solves it while still
having the host binary removed.  So it should fix the reported issue.
I also confirmed pkg-config was in the sysroot after the change.

Cc: Martin Jansa 
Signed-off-by: Paul Gortmaker 

diff --git a/meta-networking/recipes-support/netcat/netcat-openbsd_1.105.bb 
b/meta-networking/recipes-support/netcat/netcat-openbsd_1.105.bb
index bb21e0cfeede..06206a5a59d1 100644
--- a/meta-networking/recipes-support/netcat/netcat-openbsd_1.105.bb
+++ b/meta-networking/recipes-support/netcat/netcat-openbsd_1.105.bb
@@ -4,7 +4,7 @@ HOMEPAGE = "http://ftp.debian.org";
 LICENSE = "BSD-3-Clause"
 LIC_FILES_CHKSUM = 
"file://../debian/copyright;md5=ee6bbaacb5db5f2973818f0902c3ae6f"
 
-DEPENDS += "glib-2.0 libbsd"
+DEPENDS += "glib-2.0 libbsd pkgconfig-native"
 
 SRC_URI = 
"${DEBIAN_MIRROR}/main/n/netcat-openbsd/netcat-openbsd_${PV}.orig.tar.gz;name=netcat
 \

${DEBIAN_MIRROR}/main/n/netcat-openbsd/netcat-openbsd_${PV}-7.debian.tar.gz;name=netcat-patch"
@@ -42,5 +42,3 @@ do_install() {
 install -m 755 ${S}/nc ${D}${bindir}/nc.${BPN}
 }
 ALTERNATIVE_PRIORITY = "60"
-
-PNBLACKLIST[netcat-openbsd] ?= "Fails to build with RSS 
http://errors.yoctoproject.org/Errors/Details/130673/";
-- 
2.7.4

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] Splitting meta-oe?

2017-02-18 Thread Burton, Ross
On 17 February 2017 at 18:02, Martin Jansa  wrote:

> 1) RSS is good thing.
>
> 2) The breakage wasn't caused by lack of maintainers (at least I don't
>think that I or Joe were the bottleneck for integrating the fixes).
>
> 3) More maintainers doesn't mean more contributions from people actually
>using now broken components, it's actually easier to just send a fix
>than to be a maintainer of some layer just to be able to also merge
>your fix yourself.
>
> 4) It doesn't look so dramatic if it turns out that 200 of those
>blacklisted recipes weren't actually used by anyone still active in
>OE ecosystem.
>
> 5) If someone wants to replace me as meta-oe maintainer, go ahead, it
>stopped being fun for me long time ago, now it's just slightly annoying
>routine which takes my free time I would rather invest in something
>cooler
>

I'll probably reply to this properly when I'm not rushing between various
things, but I want to make it clear that I think the work you (JaMa) has
been doing on meta-oe is beyond legendary and I've no problem at all with
that.  I just wonder if it would be easier to deal with stale layers that
nobody is actually using if they were separate repositories instead of
being part of the meta-oe umbrella.

But they're not my layers so I have no say, and obviously everyone else
disagrees! :)

Ross
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [morty][meta-openembedded] Commit backport required

2017-02-18 Thread Martin Jansa
Are you sure you're using right version of bitbake? I doubt that bb.data
API changes was backported to release branch of bitbake.

On Sat, Feb 18, 2017 at 9:08 PM, Carlos Rafael Giani 
wrote:

> Hello,
>
> commit d361ef01a9a61cd8a83f7728f7d3e9f118233cb8 "nis, opencv: Update
> getVar/setVar syntax" needs to be backported to meta-openembedded's morty
> branch. Otherwise builds break, because the deprecated bb.data API was
> removed also in morty.
>
> --
> ___
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [morty][meta-openembedded] Commit backport required

2017-02-18 Thread Carlos Rafael Giani

Hello,

commit d361ef01a9a61cd8a83f7728f7d3e9f118233cb8 "nis, opencv: Update 
getVar/setVar syntax" needs to be backported to meta-openembedded's 
morty branch. Otherwise builds break, because the deprecated bb.data API 
was removed also in morty.


--
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel