From bb07d9eb0bf63e49b0f7f78b262fdf46927ed002 Mon Sep 17 00:00:00 2001 From: Richard Wang Date: Thu, 7 Jul 2022 02:28:59 +0000 Subject: [PATCH] Remove use_rbe gn arg support As it is removed in https://crrev.com/c/3744967, this cleans up the check for a no longer existing arg. Bug: chromium:1247781 Change-Id: Ic7288cf34e63cd52087e2b51f1b4c816bbbc0363 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3745201 Reviewed-by: Takuto Ikuta Reviewed-by: Bruce Dawson Commit-Queue: Richard Wang --- autoninja.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/autoninja.py b/autoninja.py index f3a3c40a4..2c53245ab 100755 --- a/autoninja.py +++ b/autoninja.py @@ -82,8 +82,7 @@ def main(args): if os.path.exists(os.path.join(output_dir, 'args.gn')): with open(os.path.join(output_dir, 'args.gn')) as file_handle: for line in file_handle: - # Either use_goma, use_remoteexec or use_rbe (in deprecation) - # activate build acceleration. + # Either use_goma or use_remoteexec will activate build acceleration. # # This test can match multi-argument lines. Examples of this are: # is_debug=false use_goma=true is_official_build=false @@ -95,7 +94,7 @@ def main(args): line_without_comment): use_goma = True continue - if re.search(r'(^|\s)(use_rbe|use_remoteexec)\s*=\s*true($|\s)', + if re.search(r'(^|\s)(use_remoteexec)\s*=\s*true($|\s)', line_without_comment): use_remoteexec = True continue