From c70446ce7b4db79f296833b16ce38eb8a01d83df Mon Sep 17 00:00:00 2001 From: Tao Bao Date: Tue, 7 Aug 2018 20:11:44 -0700 Subject: Revert "Build and use minadbd as a shared library." This reverts commit 4fd4f89591fe826cf26167bccaeb2802f14fbdc8. The reverted CL has a side effect that unintentionally installs a copy of libminadbd_services.so to the system image. This breaks the check with `build/soong/build_test.bash --dist` which detects and prevents vendor modules from installing files to the system image. Prior to fully converting `recovery` to be built with Soong, we have to statically link recovery-specific modules into `recovery`. Bug: 110380063 Test: `m -j bootimage` with aosp_marlin-userdebug Test: Run minadbd_test on marlin. Test: Boot into recovery and verify that sideloading still works. Test: `build/soong/build_test.bash --dist` Change-Id: I290d55c82e17aa60a5afdf2ff7f896afc4dae8b3 --- minadbd/Android.bp | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) (limited to 'minadbd') diff --git a/minadbd/Android.bp b/minadbd/Android.bp index 8ccce4f42..432b2f0f5 100644 --- a/minadbd/Android.bp +++ b/minadbd/Android.bp @@ -26,9 +26,8 @@ cc_defaults { ], } -cc_library { - name: "libminadbd_services", - recovery_available: true, +cc_library_static { + name: "libminadbd", defaults: [ "minadbd_defaults", @@ -40,14 +39,14 @@ cc_library { "minadbd_services.cpp", ], - shared_libs: [ - "libadbd", + static_libs: [ + "libfusesideload", "libbase", "libcrypto", ], - static_libs: [ - "libfusesideload", + whole_static_libs: [ + "libadbd", ], } @@ -63,9 +62,8 @@ cc_test { ], static_libs: [ - "libminadbd_services", - "libadbd", "libBionicGtestMain", + "libminadbd", ], shared_libs: [ -- cgit v1.2.3