diff options
| author | Steven Moreland <smoreland@google.com> | 2022-04-19 00:42:19 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2022-04-19 00:42:19 +0000 |
| commit | 4db4051c8bf8988ebaad4785799fb54da86a08af (patch) | |
| tree | 08cda715cdc47d8bc227b00027827abc69a2d4ec | |
| parent | 6350712cc08050690d42af732ad6f2386dc1db53 (diff) | |
| parent | 1f89532a35134490ff7dd610426f7d5cfc8aa797 (diff) | |
| download | aidl-4db4051c8bf8988ebaad4785799fb54da86a08af.tar.gz | |
Java: require interfaceVersion/hash for ToT vers. am: 1f89532a35
Original change: https://android-review.googlesource.com/c/platform/system/tools/aidl/+/2053109
Change-Id: I5fba08df1137b84409eb7296e91d0b69bce44c2f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | build/aidl_interface_backends.go | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/build/aidl_interface_backends.go b/build/aidl_interface_backends.go index 128eee29..62a167ce 100644 --- a/build/aidl_interface_backends.go +++ b/build/aidl_interface_backends.go @@ -81,7 +81,7 @@ func addCppLibrary(mctx android.LoadHookContext, i *aidlInterface, version strin Lang: lang, BaseName: i.ModuleBase.Name(), GenLog: genLog, - Version: i.versionForAidlGenRule(version), + Version: i.versionForInitVersionCompat(version), GenTrace: genTrace, Unstable: i.properties.Unstable, NotFrozen: notFrozen, @@ -229,7 +229,7 @@ func addJavaLibrary(mctx android.LoadHookContext, i *aidlInterface, version stri Platform_apis: proptools.Bool(i.properties.Backend.Java.Platform_apis), Lang: langJava, BaseName: i.ModuleBase.Name(), - Version: i.versionForAidlGenRule(version), + Version: version, GenRpc: proptools.Bool(i.properties.Backend.Java.Gen_rpc), GenTrace: proptools.Bool(i.properties.Gen_trace), Unstable: i.properties.Unstable, @@ -280,7 +280,7 @@ func addRustLibrary(mctx android.LoadHookContext, i *aidlInterface, version stri Min_sdk_version: i.minSdkVersion(langRust), Lang: langRust, BaseName: i.ModuleBase.Name(), - Version: i.versionForAidlGenRule(version), + Version: i.versionForInitVersionCompat(version), Unstable: i.properties.Unstable, NotFrozen: notFrozen, Flags: i.flagsForAidlGenRule(version), @@ -343,7 +343,10 @@ func (i *aidlInterface) srcsForVersion(mctx android.EarlyModuleContext, version } } -func (i *aidlInterface) versionForAidlGenRule(version string) string { +// For certain backend, avoid a difference between the initial version of a versioned +// interface and an unversioned interface. This ensures that prebuilts can't prevent +// an interface from switching from unversioned to versioned. +func (i *aidlInterface) versionForInitVersionCompat(version string) string { if !i.hasVersion() { return "" } |