Skip to content

Commit 79e7fcc

Browse files
committed
libbinder: export fewer symbols for vendor builds
See https://r.android.com/3087749 for details. This commit brings the vendor build to parity with the other builds. It may cause some initial issues for vendors, but that should be more than compensated by the ongoing issues it will fix for them. For example, when backporting security fixes, we'd often see ABI breakages because the set of internal symbols might change (before this commit we'd be exporting all internal symbols). That should never happen after this change. Bug: 338458975 Bug: 349657329 Test: m Change-Id: I5f77a8d1e16d38c5e83f9b8842368a967b4ee229
1 parent 1175227 commit 79e7fcc

1 file changed

Lines changed: 0 additions & 9 deletions

File tree

libs/binder/Android.bp

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -276,15 +276,6 @@ cc_defaults {
276276
"-fvisibility=hidden",
277277
"-DBUILDING_LIBBINDER",
278278
],
279-
280-
target: {
281-
vendor: {
282-
// Trimming the exported symbols reveals a bug in vendor code, so
283-
// disable it for the vendor variant for now. http://b/349657329
284-
// TODO: Fix the issue and remove this override.
285-
cflags: ["-fvisibility=default"],
286-
},
287-
},
288279
}
289280

290281
cc_defaults {

0 commit comments

Comments
 (0)