cros_setup_toolchain: build toolchains in reversed order.
CL:1056057 changed the dependency order between
armv7a-cros-linux-gnueabi and armv7a-cros-linux-gnueabihf toolchain.
Now armv7a-cros-linux-gnueabihf needs to be built before
armv7a-cros-linux-gnueabi, so we want to change the build order here.
CQ-DEPEND=CL:1056057
BUG=chromium:711369
TEST=chromiumos-sdk
Change-Id: Ia4cc9c8a2c87ce04f3a1fd31b7fbfc83374dd588
Reviewed-on: https://chromium-review.googlesource.com/1056058
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
diff --git a/scripts/cros_setup_toolchains.py b/scripts/cros_setup_toolchains.py
index 99344d6..b01d85c 100644
--- a/scripts/cros_setup_toolchains.py
+++ b/scripts/cros_setup_toolchains.py
@@ -263,11 +263,11 @@
cmdbase.extend(['--overlays', overlays])
cmdbase.extend(['--ov-output', CROSSDEV_OVERLAY])
- # Build target by the alphabetical order to make sure
- # armv7a-cros-linux-gnueabihf builds after armv7a-cros-linux-gnueabi
+ # Build target by the reversed alphabetical order to make sure
+ # armv7a-cros-linux-gnueabihf builds before armv7a-cros-linux-gnueabi
# because some dependency issue. This can be reverted once we
# migrated to armv7a-cros-linux-gnueabihf. crbug.com/711369
- for target in sorted(targets):
+ for target in sorted(targets, reverse=True):
if config_only and target in configured_targets:
continue