diff options
author | Paul Duffin <paulduffin@google.com> | 2022-03-18 16:57:40 +0000 |
---|---|---|
committer | Paul Duffin <paulduffin@google.com> | 2022-03-30 09:44:03 +0000 |
commit | b3805c1d357a9bef0317630a4b2463d4c3f90b0d (patch) | |
tree | 210e182afc7e86592e4c396435fde26216cb6844 /build/mainline_modules_sdks_test.py | |
parent | ec1fdcdee4d309a4532b0b11c467e3704be75f00 (diff) | |
download | common-b3805c1d357a9bef0317630a4b2463d4c3f90b0d.tar.gz |
mainline_modules_sdks.py: Optimize snapshot build order
Previously, the snapshots would be built in the order in which the
build releases were defined. This change groups the snapshots with the
same build environment and builds them in sequence in order to minimize
the regeneration of the ninja files.
The test revealed that the produce_bundled_dist_for_build_release
method would call build_snapshots even if there were no bundled
snapshots needed which was a waste so that was also optimized as part
of this change.
Test: atest --host mainline_modules_sdks_test
packages/modules/common/build/mainline_modules_sdks.sh
pyformat -s 4 --force_quote_type double -i build/mainline_modules_sdks*.py
/usr/bin/pylint --rcfile $ANDROID_BUILD_TOP/tools/repohooks/tools/pylintrc build/mainline_modules_sdks*.py
Change-Id: I5bb8d864a50318b4b5390a8e5a3601aa33c18fef
Diffstat (limited to 'build/mainline_modules_sdks_test.py')
-rw-r--r-- | build/mainline_modules_sdks_test.py | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/build/mainline_modules_sdks_test.py b/build/mainline_modules_sdks_test.py index 6ab8f280..6b77e14d 100644 --- a/build/mainline_modules_sdks_test.py +++ b/build/mainline_modules_sdks_test.py @@ -301,10 +301,8 @@ class TestProduceDist(unittest.TestCase): ["com.android.ipsec", "com.google.android.wifi"], ), ( - "S", - { - "SOONG_SDK_SNAPSHOT_TARGET_BUILD_RELEASE": "S" - }, + "latest", + {}, ["current"], [ "com.android.art", "com.android.ipsec", @@ -312,7 +310,7 @@ class TestProduceDist(unittest.TestCase): ], ), ( - "latest", + "legacy", {}, ["current"], [ @@ -320,10 +318,11 @@ class TestProduceDist(unittest.TestCase): "com.google.android.wifi" ], ), - ('latest', {}, ['current'], []), ( - "legacy", - {}, + "S", + { + "SOONG_SDK_SNAPSHOT_TARGET_BUILD_RELEASE": "S" + }, ["current"], [ "com.android.art", "com.android.ipsec", |