Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(154)

Unified Diff: tools/sublime/ninja_options_script.py

Issue 2129993002: [SublimeText] Get the clang flags with the same script that vim uses (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove extra spaces Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « docs/linux_sublime_dev.md ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/sublime/ninja_options_script.py
diff --git a/tools/sublime/ninja_options_script.py b/tools/sublime/ninja_options_script.py
index fe658da8be06d218dce8f1c67debff5474dcc455..bd93e9cd1548607ace8dd742e2ba6b9b24cababb 100755
--- a/tools/sublime/ninja_options_script.py
+++ b/tools/sublime/ninja_options_script.py
@@ -6,138 +6,39 @@
# Usage within SublimeClang:
# "sublimeclang_options_script": "python
# ${project_path}/src/tools/sublime/ninja_options_script.py \
-# ${project_path}/src \
-# ${project_path}/src/out/Debug"
+# -d '/path/to/depot_tools'"
+#
#
# NOTE: ${project_path} expands to the directory of the Sublime project file,
-# and SublimgClang passes the absolute file path to the current file as an
-# additional argument.
+# and SublimeClang passes the absolute file path to the current file as an
+# additional argument. You should change the -d argument to point to your
+# depot_tools directory.
-import fnmatch
-import logging
+import imp
+import optparse
import os
-import sys
-
-# Change to an absolute reference if ninja is not on your path
-path_to_ninja = 'ninja'
-
-# Ninja file options to extract (add 'cflags' if you need the c flags too,
-# although these usually break things)
-ninja_file_options = ['defines', 'include_dirs', 'cflags_cc']
-
-def merge_options_dicts(options_dict_1, options_dict_2):
- '''
- Given two dictionaries of options, returns one dictionary with both sets of
- options appended together.
-
- Both dictionaries must have the same options, even if some of them are empty
- lists.
- '''
- assert set(options_dict_1.keys()) == set(options_dict_2.keys()), \
- "Both options dicts must have the same keys"
- final_options_dict = {}
- for key in options_dict_1:
- final_options_dict[key] = options_dict_1[key] + options_dict_2[key]
- return final_options_dict
-
-def extract_options_from_ninja_file(ninja_file_path, options_to_extract):
- '''
- Extracts the given options from the file at ninja_file_path and returns them
- as a dictionary.
- '''
- extracted_options = dict((o, []) for o in options_to_extract)
- for line in open(ninja_file_path):
- for option in options_to_extract:
- if line.strip().startswith(option):
- extracted_options[option] += line.split('=', 1)[1].split()
- return extracted_options
-
-def find_ninja_file_options(ninja_root_path, relative_file_path_to_find,
- options_to_extract):
- '''
- Returns a dictionary of the given extracted options for the ninja file for
- relative_file_path_to_find.
-
- The options are extracted from the first *.ninja file in ninja_root_path that
- contains relative_file_path_to_find. Otherwise, the first *.ninja file that
- contains relative_file_path_to_find without the file extension. Otherwise, the
- script walks up directories until it finds ninja files and then concatenates
- the found options from all of them.
- '''
- matches = []
- for root, dirnames, filenames in os.walk(ninja_root_path):
- for filename in fnmatch.filter(filenames, '*.ninja'):
- matches.append(os.path.join(root, filename))
- logging.debug("Found %d Ninja targets", len(matches))
-
- # First, look for a *.ninja file containing the full filename.
- for ninja_file in matches:
- for line in open(ninja_file):
- if relative_file_path_to_find in line:
- return extract_options_from_ninja_file(ninja_file, options_to_extract)
-
- # Next, look for a *.ninja file containing the basename (no extension).
- # This is a fix for header files with a corresponding cpp file.
- for ninja_file in matches:
- for line in open(ninja_file):
- if os.path.splitext(relative_file_path_to_find)[0] in line:
- all_options = extract_options_from_ninja_file(ninja_file,
- options_to_extract)
- if all_options['include_dirs']:
- return all_options
-
- # Finally, open any *.ninja files in the directory or higher.
- current_path = os.path.join(ninja_root_path, 'obj',
- os.path.dirname(relative_file_path_to_find))
- while current_path != ninja_root_path:
- if os.path.exists(current_path):
- matches = []
- for root, dirnames, filenames in os.walk(ninja_root_path):
- for filename in fnmatch.filter(filenames, '*.ninja'):
- matches.append(os.path.join(root, filename))
- logging.debug("Found %d Ninja targets", len(matches))
-
- matches = []
- for match in os.listdir(current_path):
- if match.endswith('.ninja'):
- matches.append(os.path.join(current_path, match))
- all_options = dict((o, []) for o in options_to_extract)
- for ninja_file in matches:
- all_options = merge_options_dicts(all_options,
- extract_options_from_ninja_file(ninja_file, options_to_extract))
- # As soon as we have some include_dirs from the ninja files, return.
- if all_options['include_dirs']:
- return all_options
- current_path = os.path.dirname(current_path)
-
- return None
-
-project_path = sys.argv[1]
-build_path = sys.argv[2]
-file_path = sys.argv[3]
-
-logging.debug("Compiling file %s\n", file_path)
-# The file must be somewhere under the project folder...
-if not file_path.lower().startswith(project_path.lower()):
- logging.error("File %s is not in current project folder %s\n",
- file_path, project_path)
- sys.exit(1)
-file_relative_path = os.path.relpath(file_path, project_path)
-
-# Look for a .ninja file that contains our current file, since the ninja
-# project file name is needed to construct the full Ninja target path.
-logging.debug("Searching for Ninja target")
-options = find_ninja_file_options(build_path, file_relative_path,
- ninja_file_options)
-if not options:
- logging.error("File %s is not in any Ninja file under %s",
- file_relative_path, build_path)
- sys.exit(2)
-for option in ninja_file_options:
- for piece in options[option]:
- # Resolve relative includes
- if piece.startswith('-I'):
- print('-I' + os.path.join(build_path, piece[2:]))
- else:
- print(piece)
+ycm_module_path = os.path.normpath(
+ os.path.join(os.path.dirname(os.path.abspath(__file__)),
+ '../vim/chromium.ycm_extra_conf.py'))
+ycm_extra_conf = imp.load_source('ycm_extra_conf', ycm_module_path)
+
+def main():
+ usage = "usage: %prog [options] file"
+ parser = optparse.OptionParser(usage)
+ parser.add_option("-d", "--depot_tools", dest="depot_path",
+ help="path to depot_tools")
+ (options, args) = parser.parse_args()
+ if options.depot_path:
+ os.environ["PATH"] += ":%s" % options.depot_path
+ if len(args) != 1:
+ parser.error("incorrect number of arguments")
+
+ path = os.path.realpath(args[0])
+ results = ycm_extra_conf.FlagsForFile(path)
+
+ for flag in results['flags']:
+ print flag
+
+if __name__ == "__main__":
+ main()
« no previous file with comments | « docs/linux_sublime_dev.md ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698