galou created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits.
Change the default parameter `binary` in run-find-all-symbols.py to `find-all-symbols`, so that the script works out-of-the box when installed. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D84508 Files: clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py Index: clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py =================================================================== --- clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py +++ clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py @@ -66,7 +66,7 @@ parser = argparse.ArgumentParser(description='Runs find-all-symbols over all' 'files in a compilation database.') parser.add_argument('-binary', metavar='PATH', - default='./bin/find-all-symbols', + default='find-all-symbols', help='path to find-all-symbols binary') parser.add_argument('-j', type=int, default=0, help='number of instances to be run in parallel.')
Index: clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py =================================================================== --- clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py +++ clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py @@ -66,7 +66,7 @@ parser = argparse.ArgumentParser(description='Runs find-all-symbols over all' 'files in a compilation database.') parser.add_argument('-binary', metavar='PATH', - default='./bin/find-all-symbols', + default='find-all-symbols', help='path to find-all-symbols binary') parser.add_argument('-j', type=int, default=0, help='number of instances to be run in parallel.')
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits