Author: Amadeus Gebauer Date: 2023-10-20T13:40:38+02:00 New Revision: 49af6502c6dcb4a7f7520178bd14df396f78240c
URL: https://github.com/llvm/llvm-project/commit/49af6502c6dcb4a7f7520178bd14df396f78240c DIFF: https://github.com/llvm/llvm-project/commit/49af6502c6dcb4a7f7520178bd14df396f78240c.diff LOG: [run-clang-tidy] Accept export directory if PyYAML is not installed (#69700) If PyYAML is not installed, the `-export-fixes` can be used to specify a directory (not a file). Mentioning @PiotrZSL @dyung Follows #69453 Added: Modified: clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py clang-tools-extra/clang-tidy/tool/run-clang-tidy.py Removed: ################################################################################ diff --git a/clang-tools-extra/clang-tidy/tool/clang-tidy- diff .py b/clang-tools-extra/clang-tidy/tool/clang-tidy- diff .py index 6fb5eedf06d5dff..8817e2914f6e25b 100755 --- a/clang-tools-extra/clang-tidy/tool/clang-tidy- diff .py +++ b/clang-tools-extra/clang-tidy/tool/clang-tidy- diff .py @@ -187,6 +187,15 @@ def main(): "parameter is a directory, the fixes of each compilation unit are " "stored in individual yaml files in the directory.", ) + else: + parser.add_argument( + "-export-fixes", + metavar="DIRECTORY", + dest="export_fixes", + help="A directory to store suggested fixes in, which can be applied " + "with clang-apply-replacements. The fixes of each compilation unit are " + "stored in individual yaml files in the directory.", + ) parser.add_argument( "-extra-arg", dest="extra_arg", @@ -270,7 +279,12 @@ def main(): ): os.makedirs(args.export_fixes) - if not os.path.isdir(args.export_fixes) and yaml: + if not os.path.isdir(args.export_fixes): + if not yaml: + raise RuntimeError( + "Cannot combine fixes in one yaml file. Either install PyYAML or specify an output directory." + ) + combine_fixes = True if os.path.isdir(args.export_fixes): diff --git a/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py b/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py index aa628aa87800693..179759216196f88 100755 --- a/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py +++ b/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py @@ -315,6 +315,15 @@ def main(): "parameter is a directory, the fixes of each compilation unit are " "stored in individual yaml files in the directory.", ) + else: + parser.add_argument( + "-export-fixes", + metavar="directory", + dest="export_fixes", + help="A directory to store suggested fixes in, which can be applied " + "with clang-apply-replacements. The fixes of each compilation unit are " + "stored in individual yaml files in the directory.", + ) parser.add_argument( "-j", type=int, @@ -401,7 +410,12 @@ def main(): ): os.makedirs(args.export_fixes) - if not os.path.isdir(args.export_fixes) and yaml: + if not os.path.isdir(args.export_fixes): + if not yaml: + raise RuntimeError( + "Cannot combine fixes in one yaml file. Either install PyYAML or specify an output directory." + ) + combine_fixes = True if os.path.isdir(args.export_fixes): _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits