Revert "Allow visibility on libdexfile for all libdexfile_suppor..."
Revert "Add libdexfile runtime dependency of libdexfile_support."
Revert "Add libdexfile runtime dependency of libdexfile_support."
Revert submission 1810760-libdexfile-runtime-2
Reason for revert: DroidMonitor: Potential culprit for Bug 198352910 - verifying through Forrest before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.
Reverted Changes:
If4da968e4:Add libdexfile runtime dependency of libdexfile_su...
I80162942a:Allow visibility on libdexfile for all libdexfile_...
Iab18abc8e:Add libdexfile runtime dependency of libdexfile_su...
I473d146d8:Add libdexfile runtime dependency of libdexfile_su...
Change-Id: I495684a5dfe0190d2b00936830b1f24eef89a4f8
diff --git a/libdexfile/Android.bp b/libdexfile/Android.bp
index 47f2e29..5825ead 100644
--- a/libdexfile/Android.bp
+++ b/libdexfile/Android.bp
@@ -163,12 +163,7 @@
art_cc_library {
name: "libdexfile",
visibility: [
- // Allow libdexfile_support users to list this as a runtime_libs
- // dependency - see comment for libdexfile_support. It shouldn't be used
- // for any other purpose.
- "//external/perfetto",
- "//system/core/debuggerd",
- "//system/extras/simpleperf",
+ // Allow libunwindstack to list this as a runtime library.
"//system/unwinding/libunwindstack",
],
defaults: [
@@ -378,14 +373,6 @@
// dependency on dex file logic. It is therefore safe to use from binaries
// compiled without dex file support, given they won't encounter any dex file
// stack frames.
-//
-// IMPORTANT: When adding a static_libs dependency on this library, please
-// remember to also add a corresponding
-//
-// runtime_libs: ["libdexfile"],
-//
-// That is necessary since Soong doesn't propagate dependencies transitively for
-// static libraries (b/169779783).
art_cc_library_static {
name: "libdexfile_support",
visibility: ["//visibility:public"],