diff options
| author | 2022-11-14 17:35:50 +0000 | |
|---|---|---|
| committer | 2022-11-22 16:06:23 +0000 | |
| commit | c21336e7cd011ccb3bbe0f855f4a4471ffcca5c2 (patch) | |
| tree | 7076364875be4ce36b86a2809f87291ee13bdeeb /sdk/sdk_test.go | |
| parent | b452cc759f33c232d068fc6006887328a441a950 (diff) | |
Remove support for unused env vars from sdk snapshot generation
The following environment variables are no longer used when generating
sdk snapshots:
* SOONG_SDK_SNAPSHOT_PREFER - we can control the modules that are
preferred at build time now.
* SOONG_SDK_SNAPSHOT_USE_SOURCE_CONFIG_VAR - this is done as a
post-processing step by the mainline_module_sdks.sh script to avoid
having to regenerate the ninja files for every module.
* SOONG_SDK_SNAPSHOT_VERSION - support for this was removed a long time
ago but was still referenced from dumpvars.go.
Bug: 259095197
Test: m nothing
# Searched for uses of these variables but could not find any
# outside the build.
Change-Id: Id2b32b29ff6f09faffea43292f549ba13acd956d
(cherry picked from commit 82d75ade2a7dd0dbcd483c46e833ce697a443a34)
Merged-In: Id2b32b29ff6f09faffea43292f549ba13acd956d
Diffstat (limited to 'sdk/sdk_test.go')
| -rw-r--r-- | sdk/sdk_test.go | 54 |
1 files changed, 0 insertions, 54 deletions
diff --git a/sdk/sdk_test.go b/sdk/sdk_test.go index 2f9aee9da..108a664ea 100644 --- a/sdk/sdk_test.go +++ b/sdk/sdk_test.go @@ -409,60 +409,6 @@ java_import { ) }) - t.Run("SOONG_SDK_SNAPSHOT_PREFER=true", func(t *testing.T) { - result := android.GroupFixturePreparers( - preparer, - android.FixtureMergeEnv(map[string]string{ - "SOONG_SDK_SNAPSHOT_PREFER": "true", - }), - ).RunTest(t) - - checkZipFile(t, result, "out/soong/.intermediates/mysdk/common_os/mysdk-current.zip") - - CheckSnapshot(t, result, "mysdk", "", - checkAndroidBpContents(` -// This is auto-generated. DO NOT EDIT. - -java_import { - name: "myjavalib", - prefer: true, - visibility: ["//visibility:public"], - apex_available: ["//apex_available:platform"], - jars: ["java/myjavalib.jar"], -} - `), - ) - }) - - t.Run("SOONG_SDK_SNAPSHOT_USE_SOURCE_CONFIG_VAR=module:build_from_source", func(t *testing.T) { - result := android.GroupFixturePreparers( - preparer, - android.FixtureMergeEnv(map[string]string{ - "SOONG_SDK_SNAPSHOT_USE_SOURCE_CONFIG_VAR": "module:build_from_source", - }), - ).RunTest(t) - - checkZipFile(t, result, "out/soong/.intermediates/mysdk/common_os/mysdk-current.zip") - - CheckSnapshot(t, result, "mysdk", "", - checkAndroidBpContents(` -// This is auto-generated. DO NOT EDIT. - -java_import { - name: "myjavalib", - prefer: false, - use_source_config_var: { - config_namespace: "module", - var_name: "build_from_source", - }, - visibility: ["//visibility:public"], - apex_available: ["//apex_available:platform"], - jars: ["java/myjavalib.jar"], -} - `), - ) - }) - t.Run("SOONG_SDK_SNAPSHOT_TARGET_BUILD_RELEASE=S", func(t *testing.T) { result := android.GroupFixturePreparers( prepareForSdkTestWithJava, |