Re: [PATCH 3/3] eglibc-locale.inc: use nativesdk override with the PACKAGES_DYNAMIC statement

[Hongxu Jia <hongxu.jia@windriver.com>]

Hi Richard,

I have updated this patch as your suggestion, and it works well.

     nativesdk.bbclass: support nativesdk to override with the 
PACKAGES_DYNAMIC statement

     While compiling nativesdk-mtools, there was failure:
     ...
     Nothing PROVIDES 'nativesdk-glibc-gconv-ibm850'. Close matches:
     ...
     This patch supports nativesdk to override with the PACKAGES_DYNAMIC 
statement

     [YOCTO #5623]
     Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>

diff --git a/meta/classes/nativesdk.bbclass b/meta/classes/nativesdk.bbclass
index 94bc326..21a69c7 100644
--- a/meta/classes/nativesdk.bbclass
+++ b/meta/classes/nativesdk.bbclass
@@ -81,6 +81,7 @@ python () {
      clsextend.map_depends_variable("DEPENDS")
      clsextend.map_packagevars()
      clsextend.map_variable("PROVIDES")
+    clsextend.map_regexp_variable("PACKAGES_DYNAMIC")
  }

//Hongxu

On 12/03/2013 08:14 PM, Richard Purdie wrote:
> On Tue, 2013-12-03 at 18:27 +0800, Hongxu Jia wrote:
>> While compiling nativesdk-mtools, there was failure:
>> ...
>> Nothing PROVIDES 'nativesdk-glibc-gconv-ibm850'. Close matches:
>> ...
>>
>> Using nativesdk override with the PACKAGES_DYNAMIC statement fixed this issue.
>>
>> [YOCTO #5623]
>>
>> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
>> ---
>>   meta/recipes-core/eglibc/eglibc-locale.inc | 6 ++++++
>>   1 file changed, 6 insertions(+)
> No.
>
> lib/oe/classextend.py has a map_regexp_variable() function which is
> called from classes/multilib.bbclass:
>
> clsextend.map_regexp_variable("PACKAGES_DYNAMIC")
>
> Perhaps such a line should be added to nativesdk.bbclass to fix this
> properly instead?
>
> Cheers,
>
> Richard
>
>> diff --git a/meta/recipes-core/eglibc/eglibc-locale.inc b/meta/recipes-core/eglibc/eglibc-locale.inc
>> index 83569da..5e256a5 100644
>> --- a/meta/recipes-core/eglibc/eglibc-locale.inc
>> +++ b/meta/recipes-core/eglibc/eglibc-locale.inc
>> @@ -35,6 +35,12 @@ PACKAGES_DYNAMIC = "^locale-base-.* \
>>                       ^eglibc-gconv-.* ^eglibc-charmap-.* ^eglibc-localedata-.* ^eglibc-binary-localedata-.* \
>>                       ^glibc-gconv-.*  ^glibc-charmap-.*  ^glibc-localedata-.*  ^glibc-binary-localedata-.*"
>>   
>> +PACKAGES_DYNAMIC_class-nativesdk = "^nativesdk-locale-base-.* \
>> +                    ^nativesdk-eglibc-gconv-.* ^nativesdk-eglibc-charmap-.* \
>> +                    ^nativesdk-eglibc-localedata-.* ^nativesdk-eglibc-binary-localedata-.* \
>> +                    ^nativesdk-glibc-gconv-.*  ^nativesdk-glibc-charmap-.*  \
>> +                    ^nativesdk-glibc-localedata-.*  ^nativesdk-glibc-binary-localedata-.*"
>> +
>>   # Create a eglibc-binaries package
>>   ALLOW_EMPTY_${BPN}-binaries = "1"
>>   PACKAGES += "${BPN}-binaries"
>