Replace use of cronet_licenses.py with licenses.py |
||||
Issue descriptionReplace use of: //components/cronet/tools/cronet_licenses.py with //tools/licenses.py and delete cronet_licenses.py. Note that cronet_licenses.py is used for both iOS and Android builds. Need to confirm that internal GN call works fine for Android too. Context: https://codereview.chromium.org/2797403002/
,
Jun 29 2017
,
Jun 29 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/997021610f0a7631e4887e999aaacbc9cacbbce8 commit 997021610f0a7631e4887e999aaacbc9cacbbce8 Author: mef <mef@chromium.org> Date: Thu Jun 29 16:45:18 2017 [Cronet] Replace use of cronet_licenses.py with licenses.py BUG= 710801 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester Review-Url: https://codereview.chromium.org/2962963003 Cr-Commit-Position: refs/heads/master@{#483385} [modify] https://crrev.com/997021610f0a7631e4887e999aaacbc9cacbbce8/components/cronet/ios/BUILD.gn
,
Jul 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0fd5121f8fe912464b4f4d46dbb90d6209f59bf0 commit 0fd5121f8fe912464b4f4d46dbb90d6209f59bf0 Author: mef <mef@chromium.org> Date: Mon Jul 10 16:30:34 2017 [Cronet] Use gn desc to find third party licenses on Android. Filter out build/secondary/third_party dependencies to work around __main__.LicenseError: missing README.chromium or licenses.py SPECIAL_CASES entry in build/secondary/third_party/android_tools BUG= 606859 BUG= 710801 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester Review-Url: https://codereview.chromium.org/2959303002 Cr-Commit-Position: refs/heads/master@{#485291} [modify] https://crrev.com/0fd5121f8fe912464b4f4d46dbb90d6209f59bf0/components/cronet/android/BUILD.gn [modify] https://crrev.com/0fd5121f8fe912464b4f4d46dbb90d6209f59bf0/tools/licenses.py
,
Jul 20 2017
,
Jul 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/da0a4f9e5cabf886dc27ac4b7dcbd3057a42f248 commit da0a4f9e5cabf886dc27ac4b7dcbd3057a42f248 Author: Misha Efimov <mef@chromium.org> Date: Thu Jul 20 21:46:04 2017 [Cronet] Remove cronet_licenses.py replaced by common //tools/licenses.py. Bug: 710801 Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester Change-Id: I147c082f7ac74ad2bd2843b9c7ca789bac2cec7b Reviewed-on: https://chromium-review.googlesource.com/579955 Reviewed-by: Helen Li <xunjieli@chromium.org> Commit-Queue: Misha Efimov <mef@chromium.org> Cr-Commit-Position: refs/heads/master@{#488420} [delete] https://crrev.com/13ea31c035bb660596a5d15af2aed54c5e565059/components/cronet/tools/cronet_licenses.py
,
Jul 20 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by lambroslambrou@google.com
, May 2 2017