Merge changes from topic "revert-3271699-IARRNEVQHI" into main am: 6c0a5d9d30
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/3273159 Change-Id: I9ccae7ddcdd3d03291808b870b3f9339f0a1ecec Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -373,11 +373,8 @@ func (j *Javadoc) collectDeps(ctx android.ModuleContext) deps {
|
||||
panic(fmt.Errorf("unknown dependency %q for %q", otherName, ctx.ModuleName()))
|
||||
}
|
||||
case libTag, sdkLibTag:
|
||||
if _, ok := module.(SdkLibraryDependency); ok {
|
||||
sdkInfo, _ := android.OtherModuleProvider(ctx, module, SdkLibraryInfoProvider)
|
||||
generatingLibsString := android.PrettyConcat(
|
||||
getGeneratingLibs(ctx, j.SdkVersion(ctx), module.Name(), sdkInfo), true, "or")
|
||||
ctx.ModuleErrorf("cannot depend directly on java_sdk_library %q; try depending on %s instead", module.Name(), generatingLibsString)
|
||||
if dep, ok := module.(SdkLibraryDependency); ok {
|
||||
deps.classpath = append(deps.classpath, dep.SdkHeaderJars(ctx, j.SdkVersion(ctx))...)
|
||||
} else if dep, ok := android.OtherModuleProvider(ctx, module, JavaInfoProvider); ok {
|
||||
deps.classpath = append(deps.classpath, dep.HeaderJars...)
|
||||
deps.aidlIncludeDirs = append(deps.aidlIncludeDirs, dep.AidlIncludeDirs...)
|
||||
|
10
java/java.go
10
java/java.go
@@ -2700,13 +2700,13 @@ func (j *Import) GenerateAndroidBuildActions(ctx android.ModuleContext) {
|
||||
transitiveBootClasspathHeaderJars = append(transitiveBootClasspathHeaderJars, dep.TransitiveStaticLibsHeaderJars)
|
||||
}
|
||||
}
|
||||
} else if _, ok := module.(SdkLibraryDependency); ok {
|
||||
} else if dep, ok := module.(SdkLibraryDependency); ok {
|
||||
switch tag {
|
||||
case libTag, sdkLibTag:
|
||||
sdkInfo, _ := android.OtherModuleProvider(ctx, module, SdkLibraryInfoProvider)
|
||||
generatingLibsString := android.PrettyConcat(
|
||||
getGeneratingLibs(ctx, j.SdkVersion(ctx), module.Name(), sdkInfo), true, "or")
|
||||
ctx.ModuleErrorf("cannot depend directly on java_sdk_library %q; try depending on %s instead", module.Name(), generatingLibsString)
|
||||
depHeaderJars := dep.SdkHeaderJars(ctx, j.SdkVersion(ctx))
|
||||
flags.classpath = append(flags.classpath, depHeaderJars...)
|
||||
transitiveClasspathHeaderJars = append(transitiveClasspathHeaderJars,
|
||||
android.NewDepSet(android.PREORDER, depHeaderJars, nil))
|
||||
}
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user