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

Fix droiddoc handling of $(location ...)

Browse source 
am: 99e4a50202

Change-Id: I0f11abb344848b51c6fdd23345d1eef60edfe376
This commit is contained in:
Paul Duffin 2019-02-13 02:22:50 -08:00 committed by android-build-merger
commit a0957054ac
1 changed files with 12 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
java/droiddoc.go
View file

@ -695,14 +695,17 @@ func (j *Javadoc) collectDeps(ctx android.ModuleContext) deps {
j.sourcepaths = android.PathsForModuleSrc(ctx, j.properties.Local_sourcepaths)
j.argFiles = ctx.ExpandSources(j.properties.Arg_files, nil)
argFilesMap := map[string]android.Path{}
argFilesMap := map[string]string{}
argFileLabels := []string{}
for _, f := range j.argFiles {
if _, exists := argFilesMap[f.Rel()]; !exists {
argFilesMap[f.Rel()] = f
for _, label := range j.properties.Arg_files {
var paths = ctx.ExpandSources([]string{label}, nil)
if _, exists := argFilesMap[label]; !exists {
argFilesMap[label] = strings.Join(paths.Strings(), " ")
argFileLabels = append(argFileLabels, label)
} else {
ctx.ModuleErrorf("multiple arg_files for %q, %q and %q",
f, argFilesMap[f.Rel()], f.Rel())
label, argFilesMap[label], paths)
}
}
@ -710,10 +713,11 @@ func (j *Javadoc) collectDeps(ctx android.ModuleContext) deps {
j.args, err = android.Expand(String(j.properties.Args), func(name string) (string, error) {
if strings.HasPrefix(name, "location ") {
label := strings.TrimSpace(strings.TrimPrefix(name, "location "))
if f, ok := argFilesMap[label]; ok {
return f.String(), nil
if paths, ok := argFilesMap[label]; ok {
return paths, nil
} else {
return "", fmt.Errorf("unknown location label %q", label)
return "", fmt.Errorf("unknown location label %q, expecting one of %q",
label, strings.Join(argFileLabels, ", "))
}
} else if name == "genDir" {
return android.PathForModuleGen(ctx).String(), nil