diff options
author | Bill Randle <william.c.randle@intel.com> | 2016-06-24 21:22:31 -0700 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-07-01 16:22:45 +0100 |
commit | 58643b74eef6acfe4099b9f722d2a3f873623acd (patch) | |
tree | 4d46987ca6e6e8993c44be9922306553a88482c4 | |
parent | 6d2bcc247334d2e79e83e1398ac40f7ddfbf86b8 (diff) | |
download | poky-58643b74eef6acfe4099b9f722d2a3f873623acd.tar.gz |
lib/oe/package_manager: keep platform_extra and default_platform_extra lists ordered
In RpmPM:insert_feeds_uris, the paths are kept in sets, which are unordered,
but they are later used to set the priority for the Smart channels, so
unexpected results could occur. Change the sets to lists and use the same
code as in create_configs() to add items to the list, rather than the set
operators.
[YOCTO #9717]
(From OE-Core rev: ce4137f4bb955207fede0c4ef338835d9a461f59)
Signed-off-by: Bill Randle <william.c.randle@intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/lib/oe/package_manager.py | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.py index 717246d383..2004a42a01 100644 --- a/meta/lib/oe/package_manager.py +++ b/meta/lib/oe/package_manager.py | |||
@@ -700,18 +700,19 @@ class RpmPM(PackageManager): | |||
700 | arch_list = self.feed_archs.split() | 700 | arch_list = self.feed_archs.split() |
701 | else: | 701 | else: |
702 | # List must be prefered to least preferred order | 702 | # List must be prefered to least preferred order |
703 | default_platform_extra = set() | 703 | default_platform_extra = list() |
704 | platform_extra = set() | 704 | platform_extra = list() |
705 | bbextendvariant = self.d.getVar('BBEXTENDVARIANT', True) or "" | 705 | bbextendvariant = self.d.getVar('BBEXTENDVARIANT', True) or "" |
706 | for mlib in self.ml_os_list: | 706 | for mlib in self.ml_os_list: |
707 | for arch in self.ml_prefix_list[mlib]: | 707 | for arch in self.ml_prefix_list[mlib]: |
708 | plt = arch.replace('-', '_') + '-.*-' + self.ml_os_list[mlib] | 708 | plt = arch.replace('-', '_') + '-.*-' + self.ml_os_list[mlib] |
709 | if mlib == bbextendvariant: | 709 | if mlib == bbextendvariant: |
710 | default_platform_extra.add(plt) | 710 | if plt not in default_platform_extra: |
711 | default_platform_extra.append(plt) | ||
711 | else: | 712 | else: |
712 | platform_extra.add(plt) | 713 | if plt not in platform_extra: |
713 | 714 | platform_extra.append(plt) | |
714 | platform_extra = platform_extra.union(default_platform_extra) | 715 | platform_extra = default_platform_extra + platform_extra |
715 | 716 | ||
716 | for canonical_arch in platform_extra: | 717 | for canonical_arch in platform_extra: |
717 | arch = canonical_arch.split('-')[0] | 718 | arch = canonical_arch.split('-')[0] |