diff options
author | 2019-06-13 03:36:08 +0000 | |
---|---|---|
committer | 2019-06-13 03:36:08 +0000 | |
commit | e51e2feafdf7dbb84d41850237dd4858b98d93f2 (patch) | |
tree | fca1cc7df3572afabd7aefa94d98d3b53e1b2850 | |
parent | 72ea641dffd8b6d037d1529ea6946c726a624f14 (diff) | |
parent | 3245b2cb83342fa162822832d0d9adde2f0c5bed (diff) |
Merge changes I89ad92f3,Ife1604c0
* changes:
Don't add dependencies on uses-library modules in unbundled builds
Add implicit outputs to ErrorRule rules
-rw-r--r-- | android/module.go | 10 | ||||
-rw-r--r-- | java/app.go | 22 |
2 files changed, 18 insertions, 14 deletions
diff --git a/android/module.go b/android/module.go index 03993e519..87e2ca7d9 100644 --- a/android/module.go +++ b/android/module.go @@ -966,10 +966,12 @@ type moduleContext struct { func (m *moduleContext) ninjaError(params BuildParams, err error) (PackageContext, BuildParams) { return pctx, BuildParams{ - Rule: ErrorRule, - Description: params.Description, - Output: params.Output, - Outputs: params.Outputs, + Rule: ErrorRule, + Description: params.Description, + Output: params.Output, + Outputs: params.Outputs, + ImplicitOutput: params.ImplicitOutput, + ImplicitOutputs: params.ImplicitOutputs, Args: map[string]string{ "error": err.Error(), }, diff --git a/java/app.go b/java/app.go index cf9354f57..78e05012c 100644 --- a/java/app.go +++ b/java/app.go @@ -938,16 +938,18 @@ type usesLibrary struct { } func (u *usesLibrary) deps(ctx android.BottomUpMutatorContext, noFrameworkLibs bool) { - ctx.AddVariationDependencies(nil, usesLibTag, u.usesLibraryProperties.Uses_libs...) - ctx.AddVariationDependencies(nil, usesLibTag, u.presentOptionalUsesLibs(ctx)...) - if !noFrameworkLibs { - // dexpreopt/dexpreopt.go needs the paths to the dex jars of these libraries in case construct_context.sh needs - // to pass them to dex2oat. Add them as a dependency so we can determine the path to the dex jar of each - // library to dexpreopt. - ctx.AddVariationDependencies(nil, usesLibTag, - "org.apache.http.legacy", - "android.hidl.base-V1.0-java", - "android.hidl.manager-V1.0-java") + if !ctx.Config().UnbundledBuild() { + ctx.AddVariationDependencies(nil, usesLibTag, u.usesLibraryProperties.Uses_libs...) + ctx.AddVariationDependencies(nil, usesLibTag, u.presentOptionalUsesLibs(ctx)...) + if !noFrameworkLibs { + // dexpreopt/dexpreopt.go needs the paths to the dex jars of these libraries in case construct_context.sh needs + // to pass them to dex2oat. Add them as a dependency so we can determine the path to the dex jar of each + // library to dexpreopt. + ctx.AddVariationDependencies(nil, usesLibTag, + "org.apache.http.legacy", + "android.hidl.base-V1.0-java", + "android.hidl.manager-V1.0-java") + } } } |