diff options
| author | 2025-03-12 16:42:42 -0700 | |
|---|---|---|
| committer | 2025-03-12 16:42:42 -0700 | |
| commit | f2c61aac6ad79bfc9a915c303a232e90b9408db7 (patch) | |
| tree | 3cb4a723f638c9382f41e0446781ccaee2faf9e9 | |
| parent | 18548fd4d847dea2090e8763d00f4d0ee22f21d0 (diff) | |
Call SkipInstall() on all test_packages
The test package should never be installed. It's probably being
installed on some devices because it appears on the command line for
disting, and there's non-hermeticity bugs with installed files.
Bug: 402745063
Test: abtd
Change-Id: I9d2b1671fbef0ed78a4b4bafc6e858dade4bfac4
| -rw-r--r-- | ci_tests/ci_test_package_zip.go | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/ci_tests/ci_test_package_zip.go b/ci_tests/ci_test_package_zip.go index f8c4578b6..37eb625a0 100644 --- a/ci_tests/ci_test_package_zip.go +++ b/ci_tests/ci_test_package_zip.go @@ -140,6 +140,9 @@ func TestPackageZipFactory() android.Module { } func (p *testPackageZip) GenerateAndroidBuildActions(ctx android.ModuleContext) { + // Never install this test package, it's for disting only + p.SkipInstall() + if !android.InList(ctx.ModuleName(), moduleNamesAllowed) { ctx.ModuleErrorf("%s is not allowed to use module type test_package") } |