OLD | NEW |
1 #!/usr/bin/env python | 1 #!/usr/bin/env python |
2 # Copyright 2017 The Chromium Authors. All rights reserved. | 2 # Copyright 2017 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 """Find header files missing in GN. | 6 """Find header files missing in GN. |
7 | 7 |
8 This script gets all the header files from ninja_deps, which is from the true | 8 This script gets all the header files from ninja_deps, which is from the true |
9 dependency generated by the compiler, and report if they don't exist in GN. | 9 dependency generated by the compiler, and report if they don't exist in GN. |
10 """ | 10 """ |
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
69 subprocess.check_call(['gn', 'gen', out_dir, '--ide=json', '-q']) | 69 subprocess.check_call(['gn', 'gen', out_dir, '--ide=json', '-q']) |
70 gn_json = json.load(open(os.path.join(out_dir, 'project.json'))) | 70 gn_json = json.load(open(os.path.join(out_dir, 'project.json'))) |
71 q.put(ParseGNProjectJSON(gn_json)) | 71 q.put(ParseGNProjectJSON(gn_json)) |
72 | 72 |
73 | 73 |
74 def ParseGNProjectJSON(gn): | 74 def ParseGNProjectJSON(gn): |
75 """Parse GN output and get the header files""" | 75 """Parse GN output and get the header files""" |
76 all_headers = set() | 76 all_headers = set() |
77 | 77 |
78 for _target, properties in gn['targets'].iteritems(): | 78 for _target, properties in gn['targets'].iteritems(): |
79 for f in properties.get('sources', []): | 79 sources = properties.get('sources', []) |
| 80 public = properties.get('public', []) |
| 81 # Exclude '"public": "*"'. |
| 82 if type(public) is list: |
| 83 sources += public |
| 84 for f in sources: |
80 if f.endswith('.h') or f.endswith('.hh'): | 85 if f.endswith('.h') or f.endswith('.hh'): |
81 if f.startswith('//'): | 86 if f.startswith('//'): |
82 f = f[2:] # Strip the '//' prefix. | 87 f = f[2:] # Strip the '//' prefix. |
83 all_headers.add(f) | 88 all_headers.add(f) |
84 | 89 |
85 return all_headers | 90 return all_headers |
86 | 91 |
87 | 92 |
88 def GetDepsPrefixes(q): | 93 def GetDepsPrefixes(q): |
89 """Return all the folders controlled by DEPS file""" | 94 """Return all the folders controlled by DEPS file""" |
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
153 return 0 | 158 return 0 |
154 | 159 |
155 print 'The following files should be included in gn files:' | 160 print 'The following files should be included in gn files:' |
156 for i in missing: | 161 for i in missing: |
157 print i | 162 print i |
158 return 1 | 163 return 1 |
159 | 164 |
160 | 165 |
161 if __name__ == '__main__': | 166 if __name__ == '__main__': |
162 sys.exit(main()) | 167 sys.exit(main()) |
OLD | NEW |