[clang-tools-extra] [run-clang-tidy] Accept export directory if PyYAML is not installed (PR #69700)

2023-10-20 Thread Amadeus Gebauer via cfe-commits
amgebauer wrote: Is there any way to trigger the failing tests within this PR? https://github.com/llvm/llvm-project/pull/69700 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [run-clang-tidy] Accept export directory if PyYAML is not installed (PR #69700)

2023-10-20 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer edited https://github.com/llvm/llvm-project/pull/69700 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [run-clang-tidy] Accept export directory if PyYAML is not installed (PR #69700)

2023-10-20 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer created https://github.com/llvm/llvm-project/pull/69700 If PyYAML is not installed, the `-export-fixes` can be used to specify a directory (not a file). Follows #69453 >From 947b71ca6498f2783f2b701ad8d03d63779e Mon Sep 17 00:00:00 2001 From: Amadeus Gebauer D

[clang-tools-extra] [run-clang-tidy, clang-tidy-diff] Accept directory as value for -export-fixes (PR #69453)

2023-10-20 Thread Amadeus Gebauer via cfe-commits
amgebauer wrote: I have an idea about the reason. I will push a fix in a few moments. Thank you for the hint https://github.com/llvm/llvm-project/pull/69453 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman

[clang-tools-extra] [run-clang-tidy, clang-tidy-diff] Accept directory as value for -export-fixes (PR #69453)

2023-10-19 Thread Amadeus Gebauer via cfe-commits
amgebauer wrote: >From my side it is ready to be merged. I don't have the access rights to do so. https://github.com/llvm/llvm-project/pull/69453 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cf

[clang-tools-extra] [run-clang-tidy, clang-tidy-diff] Accept directory as value for -export-fixes (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
amgebauer wrote: I tested both scripts locally with both options (providing a yaml file and a directory) work on my machine. https://github.com/llvm/llvm-project/pull/69453 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org

[clang-tools-extra] [run-clang-tidy, clang-tidy-diff] Accept directory as value for -export-fixes (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer updated https://github.com/llvm/llvm-project/pull/69453 >From 182198f8ca07e458fa34158fcdbfa72310ff12f4 Mon Sep 17 00:00:00 2001 From: Amadeus Gebauer Date: Wed, 18 Oct 2023 14:02:44 +0200 Subject: [PATCH 1/2] [run-clang-tidy] Accept directory for -export-fixes ---

[clang-tools-extra] [run-clang-tidy, clang-tidy-diff] Accept directory as value for -export-fixes (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer updated https://github.com/llvm/llvm-project/pull/69453 >From 182198f8ca07e458fa34158fcdbfa72310ff12f4 Mon Sep 17 00:00:00 2001 From: Amadeus Gebauer Date: Wed, 18 Oct 2023 14:02:44 +0200 Subject: [PATCH 1/2] [run-clang-tidy] Accept directory for -export-fixes ---

[clang-tools-extra] [run-clang-tidy, clang-tidy-diff] Accept directory as value for -export-fixes (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer edited https://github.com/llvm/llvm-project/pull/69453 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [run-clang-tidy] Accept directory as value for -export-fixes (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer updated https://github.com/llvm/llvm-project/pull/69453 >From 182198f8ca07e458fa34158fcdbfa72310ff12f4 Mon Sep 17 00:00:00 2001 From: Amadeus Gebauer Date: Wed, 18 Oct 2023 14:02:44 +0200 Subject: [PATCH 1/2] [run-clang-tidy] Accept directory for -export-fixes ---

[clang-tools-extra] [run-clang-tidy] Accept directory as value for -export-fixes (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer edited https://github.com/llvm/llvm-project/pull/69453 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [run-clang-tidy] Add option -export-directory (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
amgebauer wrote: @PiotrZSL Thank you for your feedback. I implemented it the way you suggest. Feel free to leave comments. https://github.com/llvm/llvm-project/pull/69453 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/c

[clang-tools-extra] [run-clang-tidy] Add option -export-directory (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer updated https://github.com/llvm/llvm-project/pull/69453 >From d63de8e3d8e71df445983516a57cc41cf47f1a6e Mon Sep 17 00:00:00 2001 From: Amadeus Gebauer Date: Wed, 18 Oct 2023 14:02:44 +0200 Subject: [PATCH] [run-clang-tidy] Accept directory for -export-fixes --- ...

[clang-tools-extra] [run-clang-tidy] Add option -export-directory (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer edited https://github.com/llvm/llvm-project/pull/69453 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [run-clang-tidy] Add option -export-directory (PR #69453)

2023-10-18 Thread Amadeus Gebauer via cfe-commits
https://github.com/amgebauer created https://github.com/llvm/llvm-project/pull/69453 Adding an additional parameter to run_clang_tidy.py to accept a directory where the clang-tidy fixes are saved to. This directory can then be used to run `clang-apply-replacements`. >From fd908df80b14933c849e