[clang-tools-extra] [clang-tidy] Document that check_clang_tidy.py needs unix diff (PR #214681)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 7 03:44:07 PDT 2026
llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-tools-extra
Author: richardmilles
<details>
<summary>Changes</summary>
Note that failed-fix dumps use the Unix `diff` command, which must be on PATH.
Fixes #<!-- -->31242
Made with [Cursor](https://cursor.com)
---
Full diff: https://github.com/llvm/llvm-project/pull/214681.diff
1 Files Affected:
- (modified) clang-tools-extra/test/clang-tidy/check_clang_tidy.py (+4-1)
``````````diff
diff --git a/clang-tools-extra/test/clang-tidy/check_clang_tidy.py b/clang-tools-extra/test/clang-tidy/check_clang_tidy.py
index 718ae361889ce..a0f157b0a11e1 100755
--- a/clang-tools-extra/test/clang-tidy/check_clang_tidy.py
+++ b/clang-tools-extra/test/clang-tidy/check_clang_tidy.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python3
+#!/usr/bin/env python3
#
# ===-----------------------------------------------------------------------===#
#
@@ -37,6 +37,9 @@
Notes
-----
+ Failed-fix dumps use the Unix ``diff`` command (not ``git diff``); ensure
+ ``diff`` is available on PATH.
+
-std=c++(98|11|14|17|20)-or-later:
This flag will cause multiple runs within the same check_clang_tidy
execution. Make sure you don't have shared state across these runs.
``````````
</details>
https://github.com/llvm/llvm-project/pull/214681
More information about the cfe-commits
mailing list