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

Unified Diff: tools/licenses.py

Issue 2959303002: [Cronet] Use gn desc to find third party licenses on Android. (Closed)
Patch Set: Use single quotes. Created 3 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 | « components/cronet/android/BUILD.gn ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/licenses.py
diff --git a/tools/licenses.py b/tools/licenses.py
index 9f33ffb8450ef0e440658c997302b3c55a904bf1..ee65177cb96aed73d60f3617ee6df9c49b91253f 100755
--- a/tools/licenses.py
+++ b/tools/licenses.py
@@ -493,7 +493,7 @@ def GetThirdPartyDepsFromGNDepsOutput(gn_deps):
third_party_deps = set()
for build_dep in gn_deps.split():
m = re.search(r'^(.+/third_party/[^/]+)/(.+/)?BUILD\.gn$', build_dep)
- if m:
+ if m and not os.path.join('build', 'secondary') in build_dep:
third_party_deps.add(m.group(1))
return third_party_deps
« no previous file with comments | « components/cronet/android/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698