diff options
| author | 2023-10-18 22:50:26 +0000 | |
|---|---|---|
| committer | 2023-10-18 22:50:26 +0000 | |
| commit | 33d71699289ae85e9980b26b5084add84f5c920d (patch) | |
| tree | 1082642ae6a9adb51553ad85152fe95321699d4b | |
| parent | f119ceb1b2a5e46f34f92e34251795a9fc7fe6a7 (diff) | |
Revert "Revert "Add GENRULE_SANDBOXING=false to non-sandboxed bu..."
Revert submission 2794133-revert-2782270-genrule_sandboxing_by_default-JMQJWWORHR
Reason for revert: Reapplying now that ag/25093969 and ag/25107100 are in
Reverted changes: /q/submissionid:2794133-revert-2782270-genrule_sandboxing_by_default-JMQJWWORHR
Change-Id: Ibb7dd6a0f925d043ab8df2f79cbcb9d30c3be458
| -rwxr-xr-x | tests/genrule_sandbox_test.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tests/genrule_sandbox_test.py b/tests/genrule_sandbox_test.py index 0cebc2aa0..874859ad9 100755 --- a/tests/genrule_sandbox_test.py +++ b/tests/genrule_sandbox_test.py @@ -61,7 +61,8 @@ def _find_outputs_for_modules(modules, out_dir, target_product): module_path = os.path.join(out_dir, "soong", "module-actions.json") if not os.path.exists(module_path): - _build_with_soong(["json-module-graph"], target_product) + # Use GENRULE_SANDBOXING=false so that we don't cause re-analysis later when we do the no-sandboxing build + _build_with_soong(["json-module-graph"], target_product, extra_env={"GENRULE_SANDBOXING": "false"}) with open(module_path) as f: action_graph = json.load(f) @@ -131,7 +132,7 @@ def main(): all_outs = list(set.union(*module_to_outs.values())) print("building without sandboxing...") - _build_with_soong(all_outs, args.target_product) + _build_with_soong(all_outs, args.target_product, extra_env={"GENRULE_SANDBOXING": "false"}) with tempfile.TemporaryDirectory() as tempdir: for f in all_outs: subprocess.check_call(["cp", "--parents", f, tempdir]) |