From 021bf3d9733d1c985402dfb6fa82c47a37459322 Mon Sep 17 00:00:00 2001 From: Cole Faust Date: Wed, 1 May 2024 16:59:00 -0700 Subject: Reapply "Make the enabled property configurable" Previously, I had changed some loadhook-appended property structs to use selects instead of the "target" property struct. This seems to not be exactly equivalent because "target" properties are merged with the regular properties later, at the time the arch mutator runs. With this reapplication, leave those target property structs alone to avoid breakages, but I'll have to look into what the issue is with them later. This reverts commit ed5276f0827915166e89b72bf26f7e65f68d2dd5. Ignore-AOSP-First: This cl needs to be in a topic with internal-only projects, will cherrypick to aosp after. Bug: 323382414 Test: m nothing --no-skip-soong-tests Change-Id: If355d24506e3f117d27b21442a6c02bca3402dc7 --- java/hiddenapi_modular.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'java/hiddenapi_modular.go') diff --git a/java/hiddenapi_modular.go b/java/hiddenapi_modular.go index ae587eac3..cab5402e9 100644 --- a/java/hiddenapi_modular.go +++ b/java/hiddenapi_modular.go @@ -1428,7 +1428,7 @@ func deferReportingMissingBootDexJar(ctx android.ModuleContext, module android.M // should not contribute to anything. So, rather than have a missing dex jar cause a Soong // failure defer the error reporting to Ninja. Unless the prebuilt build target is explicitly // built Ninja should never use the dex jar file. - if !isActiveModule(module) { + if !isActiveModule(ctx, module) { return true } -- cgit v1.2.3-59-g8ed1b