Skip to content

Commit 754cf33

Browse files
Steven MorelandGerrit Code Review
authored andcommitted
Merge "Revert^3 "remove libbinder_ndk_helper_headers"" into main
2 parents 7153373 + 67f3674 commit 754cf33

1 file changed

Lines changed: 16 additions & 4 deletions

File tree

libs/binder/ndk/Android.bp

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,6 @@ cc_library_headers {
218218
name: "libbinder_headers_platform_shared",
219219
export_include_dirs: ["include_cpp"],
220220
vendor_available: true,
221-
product_available: true,
222221
recovery_available: true,
223222
host_supported: true,
224223
// TODO(b/153609531): remove when no longer needed.
@@ -231,9 +230,9 @@ cc_library_headers {
231230
},
232231
apex_available: [
233232
"//apex_available:platform",
234-
"//apex_available:anyapex",
233+
"com.android.media",
234+
"com.android.media.swcodec",
235235
],
236-
sdk_version: "29", // some APEXes require this, good to test headers against old SDK
237236
min_sdk_version: "29",
238237
}
239238

@@ -247,13 +246,26 @@ ndk_headers {
247246
license: "NOTICE",
248247
}
249248

250-
// include_cpp are packaged in development/build/sdk.atree with the AIDL compiler
249+
// TODO(b/160624671): package with the aidl compiler
250+
ndk_headers {
251+
name: "libbinder_ndk_helper_headers",
252+
from: "include_cpp/android",
253+
to: "android",
254+
srcs: [
255+
"include_cpp/android/*.h",
256+
],
257+
license: "NOTICE",
258+
// These are intentionally not C. It's a mistake that they're in the NDK.
259+
// See the bug above.
260+
skip_verification: true,
261+
}
251262

252263
ndk_library {
253264
name: "libbinder_ndk",
254265
symbol_file: "libbinder_ndk.map.txt",
255266
first_version: "29",
256267
export_header_libs: [
257268
"libbinder_ndk_headers",
269+
"libbinder_ndk_helper_headers",
258270
],
259271
}

0 commit comments

Comments
 (0)