diff options
author | Yifan Hong <elsk@google.com> | 2023-02-03 12:24:05 -0800 |
---|---|---|
committer | Treehugger Robot <treehugger-gerrit@google.com> | 2023-02-06 23:10:08 +0000 |
commit | 7881fb53c1716d2625ef89e2cab211f96478a791 (patch) | |
tree | 280fb0af05ccc3fa134d689eb5fbe248d6e63128 /kleaf/bazel.py | |
parent | b0a05a8e2773c542945254c140bcaaff06877e76 (diff) | |
download | build-7881fb53c1716d2625ef89e2cab211f96478a791.tar.gz |
kleaf: Allow absolute_out_dir (out/) to be overridden.
Allow to redirect the whole output directory to a different value.
Also fix previously broken test_override_javatmp test.
Test: KleafIntegrationTest.test_override_absolute_out_dir
Bug: 267580482
Change-Id: I4606567ad95537131ea043f05857e121fe2fe4bd
Diffstat (limited to 'kleaf/bazel.py')
-rwxr-xr-x | kleaf/bazel.py | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/kleaf/bazel.py b/kleaf/bazel.py index 624ce78..30bbdef 100755 --- a/kleaf/bazel.py +++ b/kleaf/bazel.py @@ -66,7 +66,6 @@ class BazelWrapper(object): self.env = env.copy() self.bazel_path = f"{self.root_dir}/{_BAZEL_REL_PATH}" - self.absolute_out_dir = f"{self.root_dir}/out" command_idx = None for idx, arg in enumerate(bazel_args): @@ -102,11 +101,15 @@ class BazelWrapper(object): """ parser = argparse.ArgumentParser(add_help=False) - parser.add_argument("--output_user_root", + parser.add_argument("--output_root", type=_require_absolute_path, - default=_require_absolute_path(f"{self.absolute_out_dir}/bazel/output_user_root")) + default=_require_absolute_path(f"{self.root_dir}/out")) + parser.add_argument("--output_user_root", + type=_require_absolute_path) known_startup_options, user_startup_options = parser.parse_known_args(self.startup_options) - self.absolute_user_root = known_startup_options.output_user_root + self.absolute_out_dir = known_startup_options.output_root + self.absolute_user_root = known_startup_options.output_user_root or \ + f"{self.absolute_out_dir}/bazel/output_user_root" self.transformed_startup_options = [ f"--host_jvm_args=-Djava.io.tmpdir={self.absolute_out_dir}/bazel/javatmp", ] |