diff --git a/git_cl.py b/git_cl.py index 200fa00970..58605c8c30 100755 --- a/git_cl.py +++ b/git_cl.py @@ -5819,6 +5819,8 @@ def CMDformat(parser, args): help='Format javascript code with clang-format.') parser.add_option('--diff', action='store_true', help='Print diff to stdout rather than modifying files.') + parser.add_option('--presubmit', action='store_true', + help='Used when running the script from a presubmit.') opts, args = parser.parse_args(args) # Normalize any remaining args against the current path, so paths relative to @@ -5961,16 +5963,21 @@ def CMDformat(parser, args): DieWithError("gn format failed on " + gn_diff_file + "\nTry running 'gn format' on this file manually.") - for xml_dir in GetDirtyMetricsDirs(diff_files): - tool_dir = os.path.join(top_dir, xml_dir) - cmd = [os.path.join(tool_dir, 'pretty_print.py'), '--non-interactive'] - if opts.dry_run or opts.diff: - cmd.append('--diff') - stdout = RunCommand(cmd, cwd=top_dir) - if opts.diff: - sys.stdout.write(stdout) - if opts.dry_run and stdout: - return_value = 2 # Not formatted. + # Skip the metrics formatting from the global presubmit hook. These files have + # a separate presubmit hook that issues an error if the files need formatting, + # whereas the top-level presubmit script merely issues a warning. Formatting + # these files is somewhat slow, so it's important not to duplicate the work. + if not opts.presubmit: + for xml_dir in GetDirtyMetricsDirs(diff_files): + tool_dir = os.path.join(top_dir, xml_dir) + cmd = [os.path.join(tool_dir, 'pretty_print.py'), '--non-interactive'] + if opts.dry_run or opts.diff: + cmd.append('--diff') + stdout = RunCommand(cmd, cwd=top_dir) + if opts.diff: + sys.stdout.write(stdout) + if opts.dry_run and stdout: + return_value = 2 # Not formatted. return return_value diff --git a/presubmit_canned_checks.py b/presubmit_canned_checks.py index 99aab04701..de4ab43c45 100644 --- a/presubmit_canned_checks.py +++ b/presubmit_canned_checks.py @@ -1088,7 +1088,7 @@ def PanProjectChecks(input_api, output_api, def CheckPatchFormatted(input_api, output_api, check_js=False): import git_cl - cmd = ['cl', 'format', '--dry-run'] + cmd = ['cl', 'format', '--dry-run', '--presubmit'] if check_js: cmd.append('--js') cmd.append(input_api.PresubmitLocalPath())