[OE-core] [PATCH 1/2] Revert "lib/oe/package_manager.py: RpmPM: fix issue with multilib builds"

Kamble, Nitin A nitin.a.kamble at intel.com
Tue Mar 4 18:37:53 UTC 2014



> -----Original Message-----
> From: openembedded-core-bounces at lists.openembedded.org
> [mailto:openembedded-core-bounces at lists.openembedded.org] On Behalf
> Of Laurentiu Palcu
> Sent: Monday, March 03, 2014 1:44 AM
> To: openembedded-core at lists.openembedded.org
> Subject: [OE-core] [PATCH 1/2] Revert "lib/oe/package_manager.py:
> RpmPM: fix issue with multilib builds"
> 
> Converting a list to a set does not keep the order of the items, as they were
> added, which might result in the wrong packages being installed in the final
> image...
> 
> This reverts commit 12f47c23df8c109676f66d580d666a3147f3b046.
> 
> [YOCTO #5890]
> 
> Signed-off-by: Laurentiu Palcu <laurentiu.palcu at intel.com>
Acked-By: Nitin A Kamble <nitin.a.kamble at intel.com>

This resolves the packaging issues I was seeing.
Thanks,
Nitin


> ---
>  meta/lib/oe/package_manager.py |   13 +++++++------
>  1 file changed, 7 insertions(+), 6 deletions(-)
> 
> diff --git a/meta/lib/oe/package_manager.py
> b/meta/lib/oe/package_manager.py index ff4f1de..6c133c3 100644
> --- a/meta/lib/oe/package_manager.py
> +++ b/meta/lib/oe/package_manager.py
> @@ -368,18 +368,19 @@ class RpmPM(PackageManager):
>                                  self.ml_os_list['default'])
> 
>          # List must be prefered to least preferred order
> -        default_platform_extra = set()
> -        platform_extra = set()
> +        default_platform_extra = list()
> +        platform_extra = list()
>          bbextendvariant = self.d.getVar('BBEXTENDVARIANT', True) or ""
>          for mlib in self.ml_os_list:
>              for arch in self.ml_prefix_list[mlib]:
>                  plt = arch.replace('-', '_') + '-.*-' + self.ml_os_list[mlib]
>                  if mlib == bbextendvariant:
> -                        default_platform_extra.add(plt)
> +                    if plt not in default_platform_extra:
> +                        default_platform_extra.append(plt)
>                  else:
> -                        platform_extra.add(plt)
> -
> -        platform_extra = platform_extra.union(default_platform_extra)
> +                    if plt not in platform_extra:
> +                        platform_extra.append(plt)
> +        platform_extra = default_platform_extra + platform_extra
> 
>          self._create_configs(platform, platform_extra)
> 
> --
> 1.7.9.5
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



More information about the Openembedded-core mailing list