From e3d1ff9fbcfd9c02e08c7bec1c8043b91adcf374 Mon Sep 17 00:00:00 2001
From: Harshit Mahajan <harshitmahajan@google.com>
Date: Fri, 19 Jul 2024 13:17:54 +0000
Subject: [CrashRecovery] Conditionally add module lib to Android SDK

Use soong select variable instead of combined_api_defaults which was not working as expected.

Bug: 349800749
Change-Id: Ie3567e43a47b62c96e2376de148cc6fa682cd4d1
Test: aninja -t path out/target/product/generic/obj/ETC/frameworks-base-api-current.txt_intermediates/frameworks-base-api-current.txt out/soong/.intermediates/packages/modules/CrashRecovery/framework/framework-crashrecovery.stubs.source/android_common/everything/framework-crashrecovery.stubs.source_api.txt
---
 api/Android.bp | 46 ++++++++++++----------------------------------
 1 file changed, 12 insertions(+), 34 deletions(-)

(limited to 'api/Android.bp')

diff --git a/api/Android.bp b/api/Android.bp
index 4de3a28db88b..4b26eb48f5de 100644
--- a/api/Android.bp
+++ b/api/Android.bp
@@ -62,40 +62,8 @@ metalava_cmd = "$(location metalava)"
 metalava_cmd += " -J--add-opens=java.base/java.util=ALL-UNNAMED "
 metalava_cmd += " --quiet "
 
-soong_config_module_type {
-    name: "enable_crashrecovery_module",
-    module_type: "combined_apis_defaults",
-    config_namespace: "ANDROID",
-    bool_variables: ["release_crashrecovery_module"],
-    properties: [
-        "bootclasspath",
-        "system_server_classpath",
-    ],
-}
-
-soong_config_bool_variable {
-    name: "release_crashrecovery_module",
-}
-
-enable_crashrecovery_module {
-    name: "crashrecovery_module_defaults",
-    soong_config_variables: {
-        release_crashrecovery_module: {
-            bootclasspath: [
-                "framework-crashrecovery",
-            ],
-            system_server_classpath: [
-                "service-crashrecovery",
-            ],
-        },
-    },
-}
-
 combined_apis {
     name: "frameworks-base-api",
-    defaults: [
-        "crashrecovery_module_defaults",
-    ],
     bootclasspath: [
         "android.net.ipsec.ike",
         "art.module.public.api",
@@ -128,7 +96,12 @@ combined_apis {
         "framework-virtualization",
         "framework-wifi",
         "i18n.module.public.api",
-    ],
+    ] + select(soong_config_variable("ANDROID", "release_crashrecovery_module"), {
+        "true": [
+            "framework-crashrecovery",
+        ],
+        default: [],
+    }),
     system_server_classpath: [
         "service-art",
         "service-configinfrastructure",
@@ -137,7 +110,12 @@ combined_apis {
         "service-permission",
         "service-rkp",
         "service-sdksandbox",
-    ],
+    ] + select(soong_config_variable("ANDROID", "release_crashrecovery_module"), {
+        "true": [
+            "service-crashrecovery",
+        ],
+        default: [],
+    }),
 }
 
 genrule {
-- 
cgit v1.2.3-59-g8ed1b