Merge "Provide some guidance for the confusing "does not provide a dex boot jar" message." am: bfaffe78de

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

Change-Id: Ie82d8878750aca1fc18b2e69aba20fb7609284f6
This commit is contained in:
Treehugger Robot
2021-12-16 17:46:23 +00:00
committed by Automerger Merge Worker

View File

@@ -506,8 +506,18 @@ func copyBootJarsToPredefinedLocations(ctx android.ModuleContext, srcBootDexJars
dst := dstBootJarsByModule[name] dst := dstBootJarsByModule[name]
if src == nil { if src == nil {
// A dex boot jar should be provided by the source java module. It needs to be installable or
// have compile_dex=true - cf. assignments to java.Module.dexJarFile.
//
// However, the source java module may be either replaced or overridden (using prefer:true) by
// a prebuilt java module with the same name. In that case the dex boot jar needs to be
// provided by the corresponding prebuilt APEX module. That APEX is the one that refers
// through a exported_(boot|systemserver)classpath_fragments property to a
// prebuilt_(boot|systemserver)classpath_fragment module, which in turn lists the prebuilt
// java module in the contents property. If that chain is broken then this dependency will
// fail.
if !ctx.Config().AllowMissingDependencies() { if !ctx.Config().AllowMissingDependencies() {
ctx.ModuleErrorf("module %s does not provide a dex boot jar", name) ctx.ModuleErrorf("module %s does not provide a dex boot jar (see comment next to this message in Soong for details)", name)
} else { } else {
ctx.AddMissingDependencies([]string{name}) ctx.AddMissingDependencies([]string{name})
} }