Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] dbus: Ensure dbus-nativesdk doesn't RPROVIDE dbus-x11
@ 2012-08-01 21:54 Richard Purdie
  2012-08-01 22:03 ` Martin Jansa
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Purdie @ 2012-08-01 21:54 UTC (permalink / raw)
  To: openembedded-core

dbus-x11 should not RPROVIDE dbus-x11 as this is incorrect and confuses
builds. This fixes the nativesdk case.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc
index 164c4d3..1984c74 100644
--- a/meta/recipes-core/dbus/dbus.inc
+++ b/meta/recipes-core/dbus/dbus.inc
@@ -33,9 +33,12 @@ DEBIANNAME_${PN} = "dbus-1"
 
 PACKAGES =+ "${PN}-lib ${PN}-systemd"
 
+OLDPKGNAME = "dbus-x11"
+OLDPKGNAME_virtclass-nativesdk = ""
+
 # for compatibility
-RPROVIDES_${PN} = "dbus-x11"
-RREPLACES_${PN} += "dbus-x11"
+RPROVIDES_${PN} = "${OLDPKGNAME}"
+RREPLACES_${PN} += "${OLDPKGNAME}"
 
 FILES_${PN}-systemd = "${systemd_unitdir}/system/"
 





^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] dbus: Ensure dbus-nativesdk doesn't RPROVIDE dbus-x11
  2012-08-01 21:54 [PATCH] dbus: Ensure dbus-nativesdk doesn't RPROVIDE dbus-x11 Richard Purdie
@ 2012-08-01 22:03 ` Martin Jansa
  2012-08-01 22:09   ` Richard Purdie
  0 siblings, 1 reply; 3+ messages in thread
From: Martin Jansa @ 2012-08-01 22:03 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 1247 bytes --]

On Wed, Aug 01, 2012 at 10:54:43PM +0100, Richard Purdie wrote:
> dbus-x11 should not RPROVIDE dbus-x11 as this is incorrect and confuses
> builds. This fixes the nativesdk case.
> 
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
> diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc
> index 164c4d3..1984c74 100644
> --- a/meta/recipes-core/dbus/dbus.inc
> +++ b/meta/recipes-core/dbus/dbus.inc
> @@ -33,9 +33,12 @@ DEBIANNAME_${PN} = "dbus-1"
>  
>  PACKAGES =+ "${PN}-lib ${PN}-systemd"
>  
> +OLDPKGNAME = "dbus-x11"
> +OLDPKGNAME_virtclass-nativesdk = ""
> +
>  # for compatibility
> -RPROVIDES_${PN} = "dbus-x11"
> -RREPLACES_${PN} += "dbus-x11"
> +RPROVIDES_${PN} = "${OLDPKGNAME}"
> +RREPLACES_${PN} += "${OLDPKGNAME}"

what about this?
http://lists.linuxtogo.org/pipermail/openembedded-core/2012-August/026947.html

>  
>  FILES_${PN}-systemd = "${systemd_unitdir}/system/"
>  
> 
> 
> 
> _______________________________________________
> 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

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] dbus: Ensure dbus-nativesdk doesn't RPROVIDE dbus-x11
  2012-08-01 22:03 ` Martin Jansa
@ 2012-08-01 22:09   ` Richard Purdie
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Purdie @ 2012-08-01 22:09 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Thu, 2012-08-02 at 00:03 +0200, Martin Jansa wrote:
> On Wed, Aug 01, 2012 at 10:54:43PM +0100, Richard Purdie wrote:
> > dbus-x11 should not RPROVIDE dbus-x11 as this is incorrect and confuses
> > builds. This fixes the nativesdk case.
> > 
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > ---
> > diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc
> > index 164c4d3..1984c74 100644
> > --- a/meta/recipes-core/dbus/dbus.inc
> > +++ b/meta/recipes-core/dbus/dbus.inc
> > @@ -33,9 +33,12 @@ DEBIANNAME_${PN} = "dbus-1"
> >  
> >  PACKAGES =+ "${PN}-lib ${PN}-systemd"
> >  
> > +OLDPKGNAME = "dbus-x11"
> > +OLDPKGNAME_virtclass-nativesdk = ""
> > +
> >  # for compatibility
> > -RPROVIDES_${PN} = "dbus-x11"
> > -RREPLACES_${PN} += "dbus-x11"
> > +RPROVIDES_${PN} = "${OLDPKGNAME}"
> > +RREPLACES_${PN} += "${OLDPKGNAME}"
> 
> what about this?
> http://lists.linuxtogo.org/pipermail/openembedded-core/2012-August/026947.html

They're each different ways to fix the problem. The above just kills the
nativesdk version since I don't think anyone will feel strongly about it
but I don't have strong feelings. I notice Marcin also has a patch out,
for some reason I didn't see that initially :(.

Cheers,

Richard




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-08-01 22:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-01 21:54 [PATCH] dbus: Ensure dbus-nativesdk doesn't RPROVIDE dbus-x11 Richard Purdie
2012-08-01 22:03 ` Martin Jansa
2012-08-01 22:09   ` Richard Purdie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox