[PATCH] distutils-base: Only RDEPEND python-core on target packages

[PATCH] distutils-base: Only RDEPEND python-core on target packages

[Martin Jansa]

* fixes ie setuptools, without this patch it RDEPENDs on python-core-native, which is not RPROVIDED by anything
* imported from OE 8377b8ec57f35b9e5b81a74c77f68fd6e02949c8

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/classes/distutils-base.bbclass |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/classes/distutils-base.bbclass b/meta/classes/distutils-base.bbclass
index 2e151de..e84b0fc 100644
--- a/meta/classes/distutils-base.bbclass
+++ b/meta/classes/distutils-base.bbclass
@@ -1,5 +1,5 @@
 DEPENDS  += "${@["python-native python", ""][(bb.data.getVar('PACKAGES', d, 1) == '')]}"
-RDEPENDS += "python-core"
+RDEPENDS_${PN} += "${@['', 'python-core']['${PN}' == '${BPN}']}"
 
 inherit distutils-common-base
 
-- 
1.7.4.1

Re: [PATCH] distutils-base: Only RDEPEND python-core on target packages

[Koen Kooi]

Op 4 mrt 2011, om 23:15 heeft Martin Jansa het volgende geschreven:

> * fixes ie setuptools, without this patch it RDEPENDs on python-core-native, which is not RPROVIDED by anything
> * imported from OE 8377b8ec57f35b9e5b81a74c77f68fd6e02949c8
> 
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
> meta/classes/distutils-base.bbclass |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/meta/classes/distutils-base.bbclass b/meta/classes/distutils-base.bbclass
> index 2e151de..e84b0fc 100644
> --- a/meta/classes/distutils-base.bbclass
> +++ b/meta/classes/distutils-base.bbclass
> @@ -1,5 +1,5 @@
> DEPENDS  += "${@["python-native python", ""][(bb.data.getVar('PACKAGES', d, 1) == '')]}"
> -RDEPENDS += "python-core"
> +RDEPENDS_${PN} += "${@['', 'python-core']['${PN}' == '${BPN}']}"

Minor nit: the two check seem to do the same, should we make them the same? 

For this patch:

Acked-by: Koen Kooi <koen@dominion.thruhere.net>

regards,

Koen

Re: [PATCH] distutils-base: Only RDEPEND python-core on target packages

[Khem Raj]

On 3/4/2011 2:41 PM, Koen Kooi wrote:
>
> Op 4 mrt 2011, om 23:15 heeft Martin Jansa het volgende geschreven:
>
>> * fixes ie setuptools, without this patch it RDEPENDs on python-core-native, which is not RPROVIDED by anything
>> * imported from OE 8377b8ec57f35b9e5b81a74c77f68fd6e02949c8
>>
>> Signed-off-by: Martin Jansa<Martin.Jansa@gmail.com>
>> ---
>> meta/classes/distutils-base.bbclass |    2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/meta/classes/distutils-base.bbclass b/meta/classes/distutils-base.bbclass
>> index 2e151de..e84b0fc 100644
>> --- a/meta/classes/distutils-base.bbclass
>> +++ b/meta/classes/distutils-base.bbclass
>> @@ -1,5 +1,5 @@
>> DEPENDS  += "${@["python-native python", ""][(bb.data.getVar('PACKAGES', d, 1) == '')]}"
>> -RDEPENDS += "python-core"
>> +RDEPENDS_${PN} += "${@['', 'python-core']['${PN}' == '${BPN}']}"
>
> Minor nit: the two check seem to do the same, should we make them the same?
>

Do you mean checks for DEPENDS and RDEPEND_${PN}

> For this patch:
>
> Acked-by: Koen Kooi<koen@dominion.thruhere.net>
>
> regards,
>
> Koen
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Re: [PATCH] distutils-base: Only RDEPEND python-core on target packages

[Koen Kooi]

Op 5 mrt 2011, om 04:19 heeft Khem Raj het volgende geschreven:

> On 3/4/2011 2:41 PM, Koen Kooi wrote:
>> 
>> Op 4 mrt 2011, om 23:15 heeft Martin Jansa het volgende geschreven:
>> 
>>> * fixes ie setuptools, without this patch it RDEPENDs on python-core-native, which is not RPROVIDED by anything
>>> * imported from OE 8377b8ec57f35b9e5b81a74c77f68fd6e02949c8
>>> 
>>> Signed-off-by: Martin Jansa<Martin.Jansa@gmail.com>
>>> ---
>>> meta/classes/distutils-base.bbclass |    2 +-
>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>> 
>>> diff --git a/meta/classes/distutils-base.bbclass b/meta/classes/distutils-base.bbclass
>>> index 2e151de..e84b0fc 100644
>>> --- a/meta/classes/distutils-base.bbclass
>>> +++ b/meta/classes/distutils-base.bbclass
>>> @@ -1,5 +1,5 @@
>>> DEPENDS  += "${@["python-native python", ""][(bb.data.getVar('PACKAGES', d, 1) == '')]}"
>>> -RDEPENDS += "python-core"
>>> +RDEPENDS_${PN} += "${@['', 'python-core']['${PN}' == '${BPN}']}"
>> 
>> Minor nit: the two check seem to do the same, should we make them the same?
>> 
> 
> Do you mean checks for DEPENDS and RDEPEND_${PN}

yes

> 
>> For this patch:
>> 
>> Acked-by: Koen Kooi<koen@dominion.thruhere.net>
>> 
>> regards,
>> 
>> Koen
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core