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
  • [PATCH] D84508: [clang-to... Gaël Écorchard via Phabricator via cfe-commits

Reply via email to