[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