[OE-core] [PATCH 00/12 V2] Fixes for multilib and eSDK

Robert Yang liezhi.yang at windriver.com
Thu Jan 18 13:31:37 UTC 2018



On 01/18/2018 07:29 PM, Richard Purdie wrote:
> On Thu, 2018-01-18 at 17:35 +0800, Robert Yang wrote:
>> Two of the patches have been merged, so I rebased others and put them
>> to:
>>
>>     git://git.openembedded.org/openembedded-core-contrib rbt/multilib
>>     http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/
>> multilib
>>
>> Robert Yang (10):
>>     staging.bbclass: fix for multilib
>>     populate_sdk_ext.bbclass: fix for multilib
>>     staging.bbclass: staging_populate_sysroot_dir(): fix for multilib
>>     staging.bbclass: print searched manifest when not found
>>     staging.bbclass: extend_recipe_sysroot(): search multilib manifest
>>     package_manager.py: reverse archs correctly
>>     multilib.bbclass: extend allarch recipes
>>     toolchain-scripts.bbclass: only install all MULTILIB_VARIANTS for
>>       image
>>     multilib.bbclass: deltask populate_sdk and populate_sdk_ext
>>     populate_sdk_ext.bbclass: remove the try...finally
> 
> Just to update, I also have a set of patches which alter the manifest
> code in staging.bbclass, I wanted to compare what we're doing and
> likely create a combination of those patches.
> 
> For the deltask commit, could you make it bb.build.deltask() only when
> image.bbclass is inherited?

Thanks, I updated it in the repo:

diff --git a/meta/classes/multilib.bbclass b/meta/classes/multilib.bbclass
index 770863f..6a79d02 100644
--- a/meta/classes/multilib.bbclass
+++ b/meta/classes/multilib.bbclass
@@ -95,6 +95,8 @@ python __anonymous () {
          d.setVar("LINGUAS_INSTALL", "")
          # FIXME, we need to map this to something, not delete it!
          d.setVar("PACKAGE_INSTALL_ATTEMPTONLY", "")
+        bb.build.deltask('do_populate_sdk', d)
+        bb.build.deltask('do_populate_sdk_ext', d)
          return

      clsextend.map_depends_variable("DEPENDS")


But I think that there is no difference since only image recipe has
do_populate_sdk/_ext.

// Robert

> 
> Cheers,
> 
> Richard
> 



More information about the Openembedded-core mailing list