[oe] [meta-oe][PATCH][master-next 3/4] pywbem: depend on python-m2crypto-native and its rdepends

[Martin Jansa]

Yes, I guessed that Richard answer was the motivation for what you did, but
the original issue reported by Andreas was about "many recipes depending on
kdoctool".

Here I see only 2 cases of such dependency, so I think we can live without
the bbclass - unless someone says that he has many recipes in internal
layers which depend on python-m2crypto-native.

Just my 2c (I'm not familiar with python-m2crypto so I cannot say how
common this use-case is).

On Mon, Feb 13, 2017 at 11:05 PM, Max Krummenacher <max.oss.09 at gmail.com>
wrote:

> Hi
>
> Am Montag, den 13.02.2017, 22:40 +0100 schrieb Martin Jansa:
> > Oh, I see what you did in master-next 1/4, but is it worth creating
> > .bbclass for just 2 dependencies which probably won't be used by so many
> > recipes?
>
> That's how I understand Richard here:
> http://lists.openembedded.org/pipermail/openembedded-core/
> 2017-February/132693.html
>
> Actually it is not 2 dependencies, it is 1 dependency on
> python-m2crypto-native and the run time dependency that
> python-m2crypto-native
> needs since it upgrade to 0.25.1.
>
> So the issue here is that if some time in the future there is another
> RDEPENDS that python-m2crypto-native needs we will have to track down
> all the users of it and add the new dependency there too.
>
> If you think this does not merit a new class I can substitute the new
> class by adding python-typing-native to crda and pywbem to fix the build
> in the current setup and resend the patch series.
>
> Max
>
>
>
> On Mon, Feb 13, 2017 at 10:39 PM, Martin Jansa <martin.jansa at gmail.com>
> > wrote:
> >
> > > Did you test this change? You added it to inherit not DEPENDS..
> > >
> > > On Mon, Feb 13, 2017 at 10:02 PM, Max Krummenacher <
> max.oss.09 at gmail.com>
> > > wrote:
> > >
> > > > This fixed the build, remove blacklist also.
> > > >
> > > > Signed-off-by: Max Krummenacher <max.krummenacher at toradex.com>
> > > > ---
> > > >  meta-oe/recipes-extended/pywbem/pywbem_0.8.0.bb | 6 +-----
> > > >  1 file changed, 1 insertion(+), 5 deletions(-)
> > > >
> > > > diff --git a/meta-oe/recipes-extended/pywbem/pywbem_0.8.0.bb
> > > > b/meta-oe/recipes-extended/pywbem/pywbem_0.8.0.bb
> > > > index 8b5394e..14d4e69 100644
> > > > --- a/meta-oe/recipes-extended/pywbem/pywbem_0.8.0.bb
> > > > +++ b/meta-oe/recipes-extended/pywbem/pywbem_0.8.0.bb
> > > > @@ -16,14 +16,13 @@ HOMEPAGE = "http://pywbem.sf.net/"
> > > >  LICENSE = "LGPLv2"
> > > >  LIC_FILES_CHKSUM = "file://pywbem/LICENSE.txt;md5
> > > > =fbc093901857fcd118f065f900982c24"
> > > >  SECTION = "Development/Libraries"
> > > > -DEPENDS = "python-m2crypto-native"
> > > >  SRC_URI = "http://jaist.dl.sourceforge.net/project/${BPN}/${BPN}/${
> BP}
> > > > /${BP}-dev.r704.zip"
> > > >  SRC_URI[md5sum] = "84072451dcdd1aa9ee82363848faf7ad"
> > > >  SRC_URI[sha256sum] = "898035866d3cc741bbcd62c4ac26e
> > > > 633ad07b7c11d89db2472b9f923f3fd3ed8"
> > > >
> > > >  S = "${WORKDIR}/${BP}-dev"
> > > >
> > > > -inherit setuptools python-dir
> > > > +inherit setuptools python-m2crypto-native
> > > >
> > > >  do_install_append() {
> > > >      mv ${D}${bindir}/wbemcli.py ${D}${bindir}/pywbemcli
> > > > @@ -34,6 +33,3 @@ do_install_append() {
> > > >  }
> > > >
> > > >  BBCLASSEXTEND = "native"
> > > > -
> > > > -PNBLACKLIST[pywbem] ?= "Fails to build with RSS
> > > > http://errors.yoctoproject.org/Errors/Details/130596/"
> > > > -PNBLACKLIST[pywbem] ?= "Fails to build with RSS
> > > > http://errors.yoctoproject.org/Errors/Details/130681/"
> > > > --
> > > > 2.6.6
> > > >
> > > > --
> > > > _______________________________________________
> > > > Openembedded-devel mailing list
> > > > Openembedded-devel at lists.openembedded.org
> > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> > > >
> > >
> > >
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>