[clang-tools-extra] [clang-tidy] Check for `-ignore-insert-conflict` support before using it in `apply_fixes` in `run-clang-tidy.py` (PR #127066)

Vicente Mataix Ferrándiz via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 17 07:10:16 PST 2025


loumalouomega wrote:

> > should have delivered a run-clang-tidy.py that is appropriate for that version.
> 
> It should, but does it? A typical installation comes from installing the Ubuntu packages, do they include `run-clang-tidy.py` at all?
> 
> If not, then I don't think `run-clang-tidy.py` should have the expectations of backwards compatibility and so on that we have for the more "public" deliveries. In that case, I agree with @whisperity that you should use a matching version.
> 
> You are not in control of the Clang/clang-tidy packages, but what about `run-clang-tidy.py`, @loumalouomega ? Can you downgrade that instead, to a version that matches the Clang/clang-tidy installation you have? I mean you should be able to clone this repo and checkout a compatible version, right? Since you already can clone the repo and put up a PR.

The script is not included in the installation, and I cannot install them, thats why I downloaded manually the script and detected the problem. I can close this, if it is too problematic. I just wanted some people to not suffer this problematic in the future.

https://github.com/llvm/llvm-project/pull/127066


More information about the cfe-commits mailing list