summaryrefslogtreecommitdiff
path: root/java/boot_jars.go
diff options
context:
space:
mode:
author Jiyong Park <jiyong@google.com> 2021-05-12 17:13:56 +0900
committer Jiyong Park <jiyong@google.com> 2021-05-18 16:05:14 +0900
commitab50b0795ac0ef5b4b6062b9d91fafbc9dda8645 (patch)
tree42315a732901675aba0cd75c9f5aed37fbb52a81 /java/boot_jars.go
parentf1d8819d7df17ab5d168f7d9d638c98b32da5765 (diff)
Rename InApexes -> InApexVariants
.. in preparation for the upcoming change. This change doesn't alter any behavior. InApexes is a misleading name. People expects that it has the list of soong module names of the APEXes that a module is part of. So, for example, `core-oj` is a part of both `com.android.art` and `com.google.android.art`. However, in reality, that's not true. The field has `com.android.art` only. This is because the two APEXes (android and Google) have the same apex name which is `com.android.art`. That apex name is used in various places like the `apex_available` and allows us to keep using the same name regardless of whether the APEX is overridden or not. However, this is causing problems in some cases where the exact list of soong module names is required. The upcoming change will add a new field to handle the case and the new field actually will get the name 'InApexes'. So, the existing field is renamed to a less misleading name `InApexVariants`. Bug: 180325915 Test: m nothing Change-Id: I0c73361b452eddb812acd5ebef5dcedaab382436
Diffstat (limited to 'java/boot_jars.go')
-rw-r--r--java/boot_jars.go2
1 files changed, 1 insertions, 1 deletions
diff --git a/java/boot_jars.go b/java/boot_jars.go
index 1fb3deb0a..1f2dab58f 100644
--- a/java/boot_jars.go
+++ b/java/boot_jars.go
@@ -89,7 +89,7 @@ func (b *bootJarsSingleton) GenerateBuildActions(ctx android.SingletonContext) {
name := android.RemoveOptionalPrebuiltPrefix(ctx.ModuleName(module))
if apex, ok := moduleToApex[name]; ok {
apexInfo := ctx.ModuleProvider(module, android.ApexInfoProvider).(android.ApexInfo)
- if (apex == "platform" && apexInfo.IsForPlatform()) || apexInfo.InApexByBaseName(apex) {
+ if (apex == "platform" && apexInfo.IsForPlatform()) || apexInfo.InApexVariantByBaseName(apex) {
// The module name/apex variant should be unique in the system but double check
// just in case something has gone wrong.
if existing, ok := nameToApexVariant[name]; ok {