From 531a416d6a35d96d709b4b3f16c96e8944dda4e3 Mon Sep 17 00:00:00 2001 From: Jihoon Kang Date: Fri, 20 Oct 2023 22:58:25 +0000 Subject: [PATCH] Modify the product variable Build_from_text_stub value productVariables.Build_from_test_stub is currently being set by the config.buildFromTextStub value. However, this leads to divergence in the behaviors between the exported BuildFromTextStub() value and the config value, as the former depends on other factor (whether is it a coverage build). This change fixes the divergence by making the product variable value to be set by the former. Test: m nothing Bug: 301522358 Change-Id: Ic4de5a179dd1094eb8788663e4d6afa4bea724ea --- android/config.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/android/config.go b/android/config.go index dfc61a5a5..8ad53e3f3 100644 --- a/android/config.go +++ b/android/config.go @@ -563,7 +563,7 @@ func NewConfig(cmdArgs CmdArgs, availableEnv map[string]string) (Config, error) config: config, } - config.productVariables.Build_from_text_stub = boolPtr(config.buildFromTextStub) + config.productVariables.Build_from_text_stub = boolPtr(config.BuildFromTextStub()) // Soundness check of the build and source directories. This won't catch strange // configurations with symlinks, but at least checks the obvious case.