Merge changes I8cf28041,If271c5f7 into main

* changes:
  Add version to stub label
  Allowlist jni_libs and transitive deps of b/297405812
This commit is contained in:
Treehugger Robot
2023-09-21 02:29:57 +00:00
committed by Gerrit Code Review
3 changed files with 10 additions and 2 deletions

View File

@@ -432,6 +432,7 @@ var (
"tools/apifinder": Bp2BuildDefaultTrue,
"tools/apksig": Bp2BuildDefaultTrue,
"tools/dexter/slicer": Bp2BuildDefaultTrueRecursively,
"tools/external_updater": Bp2BuildDefaultTrueRecursively,
"tools/metalava": Bp2BuildDefaultTrueRecursively,
"tools/platform-compat/java/android/compat": Bp2BuildDefaultTrueRecursively,
@@ -914,6 +915,11 @@ var (
"androidx.test.monitor-nodeps",
"androidx.test.annotation",
"androidx.test.annotation-nodeps",
// jni deps of an internal android_test (b/297405812)
"libdexmakerjvmtiagent",
"libopenjdkjvmti_headers",
"libstaticjvmtiagent",
}
Bp2buildModuleTypeAlwaysConvertList = []string{

View File

@@ -1635,7 +1635,7 @@ ndk_library {
],
"//build/bazel/rules/apex:unbundled_app": [
":libHasApexStubs",
"//.:libHasApexAndNdkStubs.ndk_stub_libs",
"//.:libHasApexAndNdkStubs.ndk_stub_libs-current",
],
"//conditions:default": [
":libHasApexStubs",

View File

@@ -1657,9 +1657,11 @@ func SetStubsForDynamicDeps(ctx android.BazelConversionPathContext, axis bazel.C
if depC, ok := dep.(*Module); ok && hasNdkStubs(ctx, depC) {
// If the dependency has ndk stubs, build against the ndk stubs
// https://cs.android.com/android/_/android/platform/build/soong/+/main:cc/cc.go;l=2642-2643;drc=e12d252e22dd8afa654325790d3298a0d67bd9d6;bpv=1;bpt=0
ver := proptools.String(c.Properties.Sdk_version)
// TODO - b/298085502: Add bp2build support for sdk_version: "minimum"
ndkLibModule, _ := ctx.ModuleFromName(dep.Name() + ndkLibrarySuffix)
label = bazel.Label{
Label: "//" + ctx.OtherModuleDir(ndkLibModule) + ":" + ndkLibModule.Name() + "_stub_libs",
Label: "//" + ctx.OtherModuleDir(ndkLibModule) + ":" + ndkLibModule.Name() + "_stub_libs-" + ver,
}
}
// add the ndk lib label to this axis