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

Side by Side Diff: third_party/closure_compiler/build/inputs.py

Issue 599203003: Implement recursive gyp dependencies for Closure Compilation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@J_jsevalcontext
Patch Set: store depends in list Created 6 years, 2 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 unified diff | Download patch
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 # Copyright 2014 The Chromium Authors. All rights reserved. 2 # Copyright 2014 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 import argparse 6 import argparse
7 import ast
8 import collections
7 import os 9 import os
8 import sys 10 import sys
9 11
10 12
11 sys.path.insert(0, os.path.join(os.path.dirname(__file__), "..")) 13 sys.path.insert(0, os.path.join(os.path.dirname(__file__), ".."))
12 import processor 14 import processor
13 15
14 16
17 def remove_duplicates_with_order(list_):
Dan Beam 2014/09/29 19:23:47 nit: list_ => has_dupes
Vitaly Pavlenko 2014/09/29 23:00:33 list_ => has_duplicates, done.
18 return list(collections.OrderedDict.fromkeys(list_))
19
20
21 def expand_depends(source, dep):
22 if ":" in dep:
Dan Beam 2014/09/29 19:23:47 if ":" not in dep: return [dep], {}
Vitaly Pavlenko 2014/09/29 23:00:33 Done.
23 gyp_relative_path, target = dep.split(":")
24 gyp_path = os.path.join(os.path.dirname(source), gyp_relative_path)
25 gyp_content = ast.literal_eval(open(gyp_path).read())
26
27 for target_description in gyp_content["targets"]:
28 if target_description["target_name"] == target:
29 break
30 else:
31 raise ValueError("Target '%s' not found in file '%s'" %
32 (target, gyp_path))
33
34 depends = []
35 externs = []
36 if "variables" in target_description:
37 depends = target_description["variables"].get("depends", [])
38 externs = target_description["variables"].get("externs", [])
39
40 def attach_gyp_dir(relative_path):
41 return os.path.join(os.path.dirname(gyp_path), relative_path)
42
43 target_source = attach_gyp_dir(target + ".js")
44 expanded_depends, expanded_externs = resolve_recursive_dependencies(
45 target_source,
46 depends,
47 externs)
48
49 expanded_depends = map(attach_gyp_dir, expanded_depends)
50 expanded_externs = set(map(attach_gyp_dir, expanded_externs))
51
52 expanded_depends.append(target_source)
53
54 return expanded_depends, expanded_externs
55
56 return [dep], {}
57
58
59 def resolve_recursive_dependencies(source, input_depends, depends_externs):
60 output_depends = []
61 output_externs = set(depends_externs)
62
63 for depends in input_depends:
64 expanded_depends, expanded_externs = expand_depends(source, depends)
65 output_depends.extend(expanded_depends)
66 output_externs.update(expanded_externs)
67
68 output_depends = remove_duplicates_with_order(output_depends)
69
70 return output_depends, output_externs
71
72
15 def GetInputs(args): 73 def GetInputs(args):
16 parser = argparse.ArgumentParser() 74 parser = argparse.ArgumentParser()
17 parser.add_argument("sources", nargs=argparse.ONE_OR_MORE) 75 parser.add_argument("source", nargs=1)
18 parser.add_argument("-d", "--depends", nargs=argparse.ZERO_OR_MORE, 76 parser.add_argument("-d", "--depends", nargs=argparse.ZERO_OR_MORE,
19 default=[]) 77 default=[])
20 parser.add_argument("-e", "--externs", nargs=argparse.ZERO_OR_MORE, 78 parser.add_argument("-e", "--externs", nargs=argparse.ZERO_OR_MORE,
21 default=[]) 79 default=[])
22 opts = parser.parse_args(args) 80 opts = parser.parse_args(args)
23 81
82 source = opts.source[0]
83 depends, externs = resolve_recursive_dependencies(
84 source,
85 opts.depends,
86 opts.externs)
87
24 files = set() 88 files = set()
25 for file in opts.sources + opts.depends + opts.externs: 89 for file in {source} | set(depends) | externs:
26 files.add(file) 90 files.add(file)
27 files.update(processor.Processor(file).included_files) 91 files.update(processor.Processor(file).included_files)
28 92
29 return files 93 return files
30 94
31 95
32 if __name__ == "__main__": 96 if __name__ == "__main__":
33 print "\n".join(GetInputs(sys.argv[1:])) 97 print "\n".join(GetInputs(sys.argv[1:]))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698