tequilaOS/platform_build_soong
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "Update droidstubs to support putting action inputs/outputs into the module-graph.json." am: f4bf09feba am: 23b9e124ee

Browse source 
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1940894

Change-Id: Ib40b0f4729961992da5a471eb121493c407ad6c8
This commit is contained in:
Keyi Gui 2022-01-11 04:14:23 +00:00 committed by Automerger Merge Worker
commit 9deb64462b
2 changed files with 42 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
java/droidstubs.go
View file

@ -19,6 +19,7 @@ import (
"path/filepath"
"strings"
"github.com/google/blueprint"
"github.com/google/blueprint/proptools"
"android/soong/android"
@ -806,7 +807,8 @@ type PrebuiltStubsSources struct {
properties PrebuiltStubsSourcesProperties
stubsSrcJar android.Path
stubsSrcJar android.Path
jsonDataActions []blueprint.JSONDataAction
}
func (p *PrebuiltStubsSources) OutputFiles(tag string) (android.Paths, error) {
@ -822,6 +824,13 @@ func (d *PrebuiltStubsSources) StubsSrcJar() android.Path {
return d.stubsSrcJar
}
// AddJSONData is a temporary solution for droidstubs module to put action
// related data into the module json graph.
func (p *PrebuiltStubsSources) AddJSONData(d *map[string]interface{}) {
p.ModuleBase.AddJSONData(d)
(*d)["Actions"] = blueprint.FormatJSONDataActions(p.jsonDataActions)
}
func (p *PrebuiltStubsSources) GenerateAndroidBuildActions(ctx android.ModuleContext) {
if len(p.properties.Srcs) != 1 {
ctx.PropertyErrorf("srcs", "must only specify one directory path or srcjar, contains %d paths", len(p.properties.Srcs))
@ -829,9 +838,12 @@ func (p *PrebuiltStubsSources) GenerateAndroidBuildActions(ctx android.ModuleCon
}
src := p.properties.Srcs[0]
var jsonDataAction blueprint.JSONDataAction
if filepath.Ext(src) == ".srcjar" {
// This is a srcjar. We can use it directly.
p.stubsSrcJar = android.PathForModuleSrc(ctx, src)
jsonDataAction.Inputs = []string{src}
jsonDataAction.Outputs = []string{src}
} else {
outPath := android.PathForModuleOut(ctx, ctx.ModuleName()+"-"+"stubs.srcjar")
@ -855,7 +867,10 @@ func (p *PrebuiltStubsSources) GenerateAndroidBuildActions(ctx android.ModuleCon
rule.Restat()
rule.Build("zip src", "Create srcjar from prebuilt source")
p.stubsSrcJar = outPath
jsonDataAction.Inputs = srcPaths.Strings()
jsonDataAction.Outputs = []string{outPath.String()}
}
p.jsonDataActions = []blueprint.JSONDataAction{jsonDataAction}
}
func (p *PrebuiltStubsSources) Prebuilt() *android.Prebuilt {

26
java/droidstubs_test.go
View file

@ -21,6 +21,8 @@ import (
"strings"
"testing"
"github.com/google/blueprint"
"android/soong/android"
)
@ -232,6 +234,30 @@ func TestDroidstubsWithSystemModules(t *testing.T) {
checkSystemModulesUseByDroidstubs(t, ctx, "stubs-prebuilt-system-modules", "prebuilt-jar.jar")
}
func TestAddJSONData(t *testing.T) {
prebuiltStubsSources := PrebuiltStubsSources{}
prebuiltStubsSources.jsonDataActions = []blueprint.JSONDataAction{
blueprint.JSONDataAction{
Inputs: []string{},
Outputs: []string{},
},
}
jsonData := map[string]interface{}{}
prebuiltStubsSources.AddJSONData(&jsonData)
if fmt.Sprint(jsonData) != fmt.Sprint(
map[string]interface{}{
"Android": map[string]interface{}{},
"Actions": []map[string]interface{}{
map[string]interface{}{
"Inputs": []string{},
"Outputs": []string{},
},
},
}) {
t.Errorf("The JSON data map isn't as expected %s.", jsonData)
}
}
func checkSystemModulesUseByDroidstubs(t *testing.T, ctx *android.TestContext, moduleName string, systemJar string) {
metalavaRule := ctx.ModuleForTests(moduleName, "android_common").Rule("metalava")
var systemJars []string