From 435e3b303e996904365d5e96579bfc793e113482 Mon Sep 17 00:00:00 2001 From: Olivier ROBIN Date: Mon, 28 Jul 2025 08:20:26 -0700 Subject: [PATCH] Remove Python from the command in CheckGNFormatted The command is passed directly to gn.main, so it does not need the python executable. Change-Id: I010bf68e86bf48b4b7387c26124ede216696449f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6793062 Reviewed-by: Brian Ryner Commit-Queue: Olivier Robin --- presubmit_canned_checks.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/presubmit_canned_checks.py b/presubmit_canned_checks.py index 09dfde2af1..0fd69e6679 100644 --- a/presubmit_canned_checks.py +++ b/presubmit_canned_checks.py @@ -1878,7 +1878,6 @@ def CheckGNFormatted(input_api, output_api): warnings = [] for f in affected_files: cmd = [ - input_api.python3_executable, input_api.os_path.join(_HERE, 'gn.py'), 'format', '--dry-run', f.AbsoluteLocalPath() ] @@ -2864,4 +2863,4 @@ def CheckValidHostsInDEPSOnUpload(input_api, output_api): output_api.PresubmitError( 'DEPS file must have only git dependencies.', long_text=error.output) - ] \ No newline at end of file + ]