Patchwork [1/2] python-dbus: Native dbus python module needed for dbus tests

login
register
mail settings
Submitter Andrei Gherzan
Date Aug. 17, 2012, 1:05 p.m.
Message ID <f3b626bf0865fc3fbccb898723f162f900aa244f.1345208249.git.andrei.gherzan@windriver.com>
Download mbox | patch
Permalink /patch/34819/
State New
Headers show

Comments

Andrei Gherzan - Aug. 17, 2012, 1:05 p.m.
Signed-off-by: Andrei Gherzan <andrei.gherzan@windriver.com>
---
 meta/recipes-devtools/python/python-dbus_1.1.1.bb |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)
Martin Jansa - Aug. 17, 2012, 1:09 p.m.
On Fri, Aug 17, 2012 at 04:05:00PM +0300, Andrei Gherzan wrote:
> Signed-off-by: Andrei Gherzan <andrei.gherzan@windriver.com>
> ---
>  meta/recipes-devtools/python/python-dbus_1.1.1.bb |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/meta/recipes-devtools/python/python-dbus_1.1.1.bb b/meta/recipes-devtools/python/python-dbus_1.1.1.bb
> index 51d5bc2..35149a0 100644
> --- a/meta/recipes-devtools/python/python-dbus_1.1.1.bb
> +++ b/meta/recipes-devtools/python/python-dbus_1.1.1.bb
> @@ -4,7 +4,7 @@ HOMEPAGE = "http://www.freedesktop.org/Software/dbus"
>  LICENSE = "MIT"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=f5612614133e9a2f2dad527d97554670"
>  DEPENDS = "expat dbus dbus-glib virtual/libintl python-pyrex-native"
> -PR = "r0"
> +PR = "r1"
>  
>  SRC_URI = "http://dbus.freedesktop.org/releases/dbus-python/dbus-python-${PV}.tar.gz"
>  
> @@ -23,3 +23,6 @@ export STAGING_INCDIR
>  RDEPENDS_${PN} = "python-io python-logging python-stringold python-threading python-xml"
>  
>  FILES_${PN}-dev += "${libdir}/pkgconfig"
> +
> +RDEPENDS_${PN}_virtclass-native = "python"

Does RDEPENDS_${PN} work for native recipes? I thought that R* vars are
for runtime packages and native builds doesn't provide them imho.

And if they do, why not explicitly ask for python-native, because target python wont help I guess.

> +BBCLASSEXTEND = "native"
> -- 
> 1.7.5.4
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
Andrei Gherzan - Aug. 17, 2012, 1:55 p.m.
On Fri, Aug 17, 2012 at 4:09 PM, Martin Jansa <martin.jansa@gmail.com>wrote:

> On Fri, Aug 17, 2012 at 04:05:00PM +0300, Andrei Gherzan wrote:
> > Signed-off-by: Andrei Gherzan <andrei.gherzan@windriver.com>
> > ---
> >  meta/recipes-devtools/python/python-dbus_1.1.1.bb |    5 ++++-
> >  1 files changed, 4 insertions(+), 1 deletions(-)
> >
> > diff --git a/meta/recipes-devtools/python/python-dbus_1.1.1.bbb/meta/recipes-devtools/python/
> python-dbus_1.1.1.bb
> > index 51d5bc2..35149a0 100644
> > --- a/meta/recipes-devtools/python/python-dbus_1.1.1.bb
> > +++ b/meta/recipes-devtools/python/python-dbus_1.1.1.bb
> > @@ -4,7 +4,7 @@ HOMEPAGE = "http://www.freedesktop.org/Software/dbus"
> >  LICENSE = "MIT"
> >  LIC_FILES_CHKSUM = "file://COPYING;md5=f5612614133e9a2f2dad527d97554670"
> >  DEPENDS = "expat dbus dbus-glib virtual/libintl python-pyrex-native"
> > -PR = "r0"
> > +PR = "r1"
> >
> >  SRC_URI = "
> http://dbus.freedesktop.org/releases/dbus-python/dbus-python-${PV}.tar.gz"
> >
> > @@ -23,3 +23,6 @@ export STAGING_INCDIR
> >  RDEPENDS_${PN} = "python-io python-logging python-stringold
> python-threading python-xml"
> >
> >  FILES_${PN}-dev += "${libdir}/pkgconfig"
> > +
> > +RDEPENDS_${PN}_virtclass-native = "python"
>
> Does RDEPENDS_${PN} work for native recipes? I thought that R* vars are
> for runtime packages and native builds doesn't provide them imho.
>
> And if they do, why not explicitly ask for python-native, because target
> python wont help I guess.
>
>
The idea here was that rdepends should be overwritten as - python-io
python-logging python-stringold python-threading python-xml - are not
native packages. While compiling (python-dbus-native) build-system is
looking for python-io-native for example which wrong and fails:
ERROR: Nothing RPROVIDES 'python-io-native'

Do you think of a better solution?

Thank you


>  > +BBCLASSEXTEND = "native"
> > --
> > 1.7.5.4
> >
> >
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
> --
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>

Patch

diff --git a/meta/recipes-devtools/python/python-dbus_1.1.1.bb b/meta/recipes-devtools/python/python-dbus_1.1.1.bb
index 51d5bc2..35149a0 100644
--- a/meta/recipes-devtools/python/python-dbus_1.1.1.bb
+++ b/meta/recipes-devtools/python/python-dbus_1.1.1.bb
@@ -4,7 +4,7 @@  HOMEPAGE = "http://www.freedesktop.org/Software/dbus"
 LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://COPYING;md5=f5612614133e9a2f2dad527d97554670"
 DEPENDS = "expat dbus dbus-glib virtual/libintl python-pyrex-native"
-PR = "r0"
+PR = "r1"
 
 SRC_URI = "http://dbus.freedesktop.org/releases/dbus-python/dbus-python-${PV}.tar.gz"
 
@@ -23,3 +23,6 @@  export STAGING_INCDIR
 RDEPENDS_${PN} = "python-io python-logging python-stringold python-threading python-xml"
 
 FILES_${PN}-dev += "${libdir}/pkgconfig"
+
+RDEPENDS_${PN}_virtclass-native = "python"
+BBCLASSEXTEND = "native"