From f0f2e2cf79558569c227e8d96a784d9c0679814a Mon Sep 17 00:00:00 2001 From: Colin Cross Date: Tue, 15 Oct 2019 16:36:40 -0700 Subject: [PATCH] Add tradefed_java_library_host Add a module type for tradefed libraries that causes an additional copy to be installed at out/host/linux-x86/tradefed. Bug: 122332235 Bug: 141877526 Test: m tradefed Change-Id: Ib7160dd060ad83d2dfb81d6a7106cb0dac98553e --- Android.bp | 1 + java/java.go | 8 +++++++- java/tradefed.go | 37 +++++++++++++++++++++++++++++++++++++ 3 files changed, 45 insertions(+), 1 deletion(-) create mode 100644 java/tradefed.go diff --git a/Android.bp b/Android.bp index dd538185b..d06b4598d 100644 --- a/Android.bp +++ b/Android.bp @@ -294,6 +294,7 @@ bootstrap_go_package { "java/support_libraries.go", "java/system_modules.go", "java/testing.go", + "java/tradefed.go", ], testSrcs: [ "java/androidmk_test.go", diff --git a/java/java.go b/java/java.go index 3b95f1e47..13fce085b 100644 --- a/java/java.go +++ b/java/java.go @@ -1571,6 +1571,8 @@ func (j *Module) hasCode(ctx android.ModuleContext) bool { type Library struct { Module + + InstallMixin func(ctx android.ModuleContext, installPath android.Path) (extraInstallDeps android.Paths) } func shouldUncompressDex(ctx android.ModuleContext, dexpreopter *dexpreopter) bool { @@ -1601,8 +1603,12 @@ func (j *Library) GenerateAndroidBuildActions(ctx android.ModuleContext) { exclusivelyForApex := android.InAnyApex(ctx.ModuleName()) && !j.IsForPlatform() if (Bool(j.properties.Installable) || ctx.Host()) && !exclusivelyForApex { + var extraInstallDeps android.Paths + if j.InstallMixin != nil { + extraInstallDeps = j.InstallMixin(ctx, j.outputFile) + } j.installFile = ctx.InstallFile(android.PathForModuleInstall(ctx, "framework"), - ctx.ModuleName()+".jar", j.outputFile) + ctx.ModuleName()+".jar", j.outputFile, extraInstallDeps...) } } diff --git a/java/tradefed.go b/java/tradefed.go new file mode 100644 index 000000000..ebbdec13d --- /dev/null +++ b/java/tradefed.go @@ -0,0 +1,37 @@ +// Copyright 2019 Google Inc. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +package java + +import ( + "android/soong/android" +) + +func init() { + android.RegisterModuleType("tradefed_java_library_host", tradefedJavaLibraryFactory) +} + +// tradefed_java_library_factory wraps java_library and installs an additional +// copy of the output jar to $HOST_OUT/tradefed. +func tradefedJavaLibraryFactory() android.Module { + module := LibraryHostFactory().(*Library) + module.InstallMixin = tradefedJavaLibraryInstall + return module +} + +func tradefedJavaLibraryInstall(ctx android.ModuleContext, path android.Path) android.Paths { + installedPath := ctx.InstallFile(android.PathForModuleInstall(ctx, "tradefed"), + ctx.ModuleName()+".jar", path) + return android.Paths{installedPath} +}