commit 4f66ef89981daba36f1076b0aac223c68b646044
author Martin Stjernholm <mast@google.com> Wed Oct 23 21:52:29 2019 +0100
committer TreeHugger Robot <treehugger-gerrit@google.com> Wed Mar 25 20:29:25 2020 +0000
tree ee66dcd0fdc3c41f0df05b5a9193ac5260425420
parent c0f722184559018b6ad201f261f2f8fa224aebcf

Remove access to internal dex libs for com.android.runtime.

Test: Build & boot
Bug: 142944931
Bug: 151466650
Change-Id: I2844d372c0d46ec8dffcbad1812a6ea944c6fe9c
Merged-In: I2844d372c0d46ec8dffcbad1812a6ea944c6fe9c

libartbase/Android.bp

diff --git a/libartbase/Android.bp b/libartbase/Android.bp
index a9f9918..c110315 100644
--- a/libartbase/Android.bp
+++ b/libartbase/Android.bp
@@ -176,11 +176,6 @@
     apex_available: [
         "com.android.art.release",
         "com.android.art.debug",
-        // due to the transitive dependency from linker:
-        // linker -> libdebuggerd_handler_fallback -> libdexfile_support_static -> libdexfile
-        // libartbase
-        // TODO(b/142944931): remove this
-        "com.android.runtime",
         // TODO(b/142944931) Clean this up. This is due to the dependency to libdexfile_support_static
         // from /system/core, etc.
         "//apex_available:platform",

libdexfile/Android.bp

diff --git a/libdexfile/Android.bp b/libdexfile/Android.bp
index 626295c..a36a859 100644
--- a/libdexfile/Android.bp
+++ b/libdexfile/Android.bp
@@ -177,8 +177,6 @@
     apex_available: [
         "com.android.art.release",
         "com.android.art.debug",
-        // TODO(b/142944931): remove this
-        "com.android.runtime", // due to the transitive dependency from linker
         // TODO(b/142944931) Clean this up. This is due to the dependency to libdexfile_support_static
         // from /system/core, etc.
         "//apex_available:platform",
@@ -313,8 +311,6 @@
     apex_available: [
         "com.android.art.release",
         "com.android.art.debug",
-        // TODO(b/142944931): remove this
-        "com.android.runtime", // due to the transitive dependency from linker
         // TODO(b/142944931) Clean this up. This is due to the dependency to libdexfile_support_static
         // from /system/core, etc.
         "//apex_available:platform",
@@ -422,9 +418,8 @@
     name: "libdexfile_support_static",
     visibility: [
         "//art:__subpackages__",
-        // TODO(b/133140750): Clean this up.
-        "//system/core/debuggerd",
-        "//system/core/init", // Dependency from Android.mk file, so not enforced.
+        // Required for the simpleperf binary in the NDK. No other modules than
+        // //system/extras/simpleperf:simpleperf_ndk are allowed to use it.
         "//system/extras/simpleperf",
     ],
     defaults: [