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 <[email protected]> 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 <[email protected]> > > wrote: > > > > > Did you test this change? You added it to inherit not DEPENDS.. > > > > > > On Mon, Feb 13, 2017 at 10:02 PM, Max Krummenacher < > [email protected]> > > > wrote: > > > > > > > This fixed the build, remove blacklist also. > > > > > > > > Signed-off-by: Max Krummenacher <[email protected]> > > > > --- > > > > 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 > > > > [email protected] > > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > > > > > > > > > > -- > _______________________________________________ > Openembedded-devel mailing list > [email protected] > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > -- _______________________________________________ Openembedded-devel mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-devel
