Patchwork dbus.inc: s/INC_PR/PR/ to fix PR bump after dbus-launch-helper permission change

login
register
mail settings
Submitter Koen Kooi
Date Jan. 12, 2012, 11:52 a.m.
Message ID <1326369146-4369-1-git-send-email-koen@dominion.thruhere.net>
Download mbox | patch
Permalink /patch/19161/
State New
Headers show

Comments

Koen Kooi - Jan. 12, 2012, 11:52 a.m.
The dbus recipes don't use INC_PR so it didn't get rebuilt after the previous commit to fix the launch helper permissions

Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
 meta/recipes-core/dbus/dbus.inc |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Martin Jansa - Jan. 12, 2012, 12:19 p.m.
On Thu, Jan 12, 2012 at 12:52:26PM +0100, Koen Kooi wrote:
> The dbus recipes don't use INC_PR so it didn't get rebuilt after the previous commit to fix the launch helper permissions
> 
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
>  meta/recipes-core/dbus/dbus.inc |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc
> index 2a25256..489e8db 100644
> --- a/meta/recipes-core/dbus/dbus.inc
> +++ b/meta/recipes-core/dbus/dbus.inc
> @@ -10,7 +10,7 @@ DEPENDS = "expat virtual/libintl ${@base_contains('DISTRO_FEATURES', 'x11', '${X
>  DEPENDS_virtclass-native = "expat-native virtual/libintl-native"
>  DEPENDS_virtclass-nativesdk = "expat-nativesdk virtual/libintl-nativesdk virtual/libx11"
>  
> -INC_PR = "r1"
> +PR = "r1"

Can we move it to dbus_1.4.16.bb too? or change dbus_1.4.16.bb to use
INC_PR, I can imagine someone reusing meta/recipes-core/dbus/dbus.inc in
his layer won't notice that there is PR now.

Cheers,

>  
>  SRC_URI = "http://dbus.freedesktop.org/releases/dbus/dbus-${PV}.tar.gz \
>             file://tmpdir.patch; \
> -- 
> 1.7.2.5
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
Koen Kooi - Jan. 12, 2012, 1:34 p.m.
Op 12 jan. 2012, om 13:19 heeft Martin Jansa het volgende geschreven:

> On Thu, Jan 12, 2012 at 12:52:26PM +0100, Koen Kooi wrote:
>> The dbus recipes don't use INC_PR so it didn't get rebuilt after the previous commit to fix the launch helper permissions
>> 
>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
>> ---
>> meta/recipes-core/dbus/dbus.inc |    2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>> 
>> diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc
>> index 2a25256..489e8db 100644
>> --- a/meta/recipes-core/dbus/dbus.inc
>> +++ b/meta/recipes-core/dbus/dbus.inc
>> @@ -10,7 +10,7 @@ DEPENDS = "expat virtual/libintl ${@base_contains('DISTRO_FEATURES', 'x11', '${X
>> DEPENDS_virtclass-native = "expat-native virtual/libintl-native"
>> DEPENDS_virtclass-nativesdk = "expat-nativesdk virtual/libintl-nativesdk virtual/libx11"
>> 
>> -INC_PR = "r1"
>> +PR = "r1"
> 
> Can we move it to dbus_1.4.16.bb too? or change dbus_1.4.16.bb to use
> INC_PR, I can imagine someone reusing meta/recipes-core/dbus/dbus.inc in
> his layer won't notice that there is PR now.

I don't mind either way, I just want the fixed dbus package in the binary feeds :)

Patch

diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc
index 2a25256..489e8db 100644
--- a/meta/recipes-core/dbus/dbus.inc
+++ b/meta/recipes-core/dbus/dbus.inc
@@ -10,7 +10,7 @@  DEPENDS = "expat virtual/libintl ${@base_contains('DISTRO_FEATURES', 'x11', '${X
 DEPENDS_virtclass-native = "expat-native virtual/libintl-native"
 DEPENDS_virtclass-nativesdk = "expat-nativesdk virtual/libintl-nativesdk virtual/libx11"
 
-INC_PR = "r1"
+PR = "r1"
 
 SRC_URI = "http://dbus.freedesktop.org/releases/dbus/dbus-${PV}.tar.gz \
            file://tmpdir.patch; \