tools/licenses.py uses a harcoded gn location |
||
Issue descriptionhttps://logs.chromium.org/v/?s=chromium%2Fbuildbucket%2Fcr-buildbucket.appspot.com%2F8942598860723933344%2F%2B%2Fsteps%2Fcompile__with_patch_%2F0%2Fstdout FAILED: cronet/LICENSE python ../../tools/licenses.py license_file cronet/LICENSE --gn-target //components/cronet/android:cronet --gn-out-dir . Traceback (most recent call last): File "../../tools/licenses.py", line 757, in <module> sys.exit(main()) File "../../tools/licenses.py", line 749, in main args.output_file, args.gn_out_dir, args.gn_target): File "../../tools/licenses.py", line 694, in GenerateLicenseFile third_party_dirs = FindThirdPartyDeps(gn_out_dir, gn_target) File "../../tools/licenses.py", line 529, in FindThirdPartyDeps subprocess.check_output([_GnBinary(), "gen", tmp_dir]) File "/b/swarming/w/ir/cipd_bin_packages/lib/python2.7/subprocess.py", line 212, in check_output process = Popen(stdout=PIPE, *popenargs, **kwargs) File "/b/swarming/w/ir/cipd_bin_packages/lib/python2.7/subprocess.py", line 390, in __init__ errread, errwrite) File "/b/swarming/w/ir/cipd_bin_packages/lib/python2.7/subprocess.py", line 1025, in _execute_child raise child_exception OSError: [Errno 2] No such file or directory
,
Jun 27 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0895c7977a5c312aca132b23a3a9d46f78f77ee4 commit 0895c7977a5c312aca132b23a3a9d46f78f77ee4 Author: Scott Graham <scottmg@chromium.org> Date: Wed Jun 27 16:10:36 2018 Pull GN via CIPD package The gn binary will be downloaded into third_party/gn. This CL rolls buildtools forward to a revision where gn is no longer in buildtools. Because buildtools has no way to cleanup after itself, add build/util/gn_cleanup.py to delete the old binaries so that we don't accidentally depend on them. Update mb.py to use the new location. Update explicit location in ios/web_view/BUILD.gn for the new location. Update explicit location in tools/traffic_annotation/auditor/traffic_annotation_auditor.cc for the new location. Update explicit location in tools/licenses.py with for the new location. Bug: 855791 Bug: 794764 Bug: 856883 Bug: 856884 Bug: 856878 Bug: 856899 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I92c908faf4f868850eafa1b4adf6e7c33c365116 Reviewed-on: https://chromium-review.googlesource.com/1112840 Commit-Queue: Scott Graham <scottmg@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#570792} [modify] https://crrev.com/0895c7977a5c312aca132b23a3a9d46f78f77ee4/DEPS [add] https://crrev.com/0895c7977a5c312aca132b23a3a9d46f78f77ee4/build/util/gn_cleanup.py [modify] https://crrev.com/0895c7977a5c312aca132b23a3a9d46f78f77ee4/ios/web_view/BUILD.gn [modify] https://crrev.com/0895c7977a5c312aca132b23a3a9d46f78f77ee4/tools/licenses.py [modify] https://crrev.com/0895c7977a5c312aca132b23a3a9d46f78f77ee4/tools/mb/mb.py [modify] https://crrev.com/0895c7977a5c312aca132b23a3a9d46f78f77ee4/tools/mb/mb_unittest.py [modify] https://crrev.com/0895c7977a5c312aca132b23a3a9d46f78f77ee4/tools/traffic_annotation/auditor/traffic_annotation_auditor.cc
,
Jun 27 2018
,
Jun 27 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/35383539aa83f27185a661215773ece946535676 commit 35383539aa83f27185a661215773ece946535676 Author: Scott Graham <scottmg@chromium.org> Date: Wed Jun 27 16:27:35 2018 Revert "Pull GN via CIPD package" This reverts commit 0895c7977a5c312aca132b23a3a9d46f78f77ee4. Reason for revert: Missed a hardcoded gn location: https://crbug.com/857107. Original change's description: > Pull GN via CIPD package > > The gn binary will be downloaded into third_party/gn. This CL rolls > buildtools forward to a revision where gn is no longer in buildtools. > > Because buildtools has no way to cleanup after itself, add > build/util/gn_cleanup.py to delete the old binaries so that we don't > accidentally depend on them. > > Update mb.py to use the new location. > > Update explicit location in ios/web_view/BUILD.gn for the new location. > > Update explicit location in > tools/traffic_annotation/auditor/traffic_annotation_auditor.cc for the new > location. > > Update explicit location in tools/licenses.py with for the new location. > > Bug: 855791 > Bug: 794764 > Bug: 856883 > Bug: 856884 > Bug: 856878 > Bug: 856899 > Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet > Change-Id: I92c908faf4f868850eafa1b4adf6e7c33c365116 > Reviewed-on: https://chromium-review.googlesource.com/1112840 > Commit-Queue: Scott Graham <scottmg@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Cr-Commit-Position: refs/heads/master@{#570792} TBR=dpranke@chromium.org,scottmg@chromium.org Change-Id: I89e180710b5ce483106fa3b4ff6e8dec06780c07 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 855791, 794764 , 856883, 856884 , 856878, 856899 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Reviewed-on: https://chromium-review.googlesource.com/1117219 Reviewed-by: Scott Graham <scottmg@chromium.org> Commit-Queue: Scott Graham <scottmg@chromium.org> Cr-Commit-Position: refs/heads/master@{#570799} [modify] https://crrev.com/35383539aa83f27185a661215773ece946535676/DEPS [delete] https://crrev.com/d0ed5774569fc56f4a0c74affafda8a7ad2a02aa/build/util/gn_cleanup.py [modify] https://crrev.com/35383539aa83f27185a661215773ece946535676/ios/web_view/BUILD.gn [modify] https://crrev.com/35383539aa83f27185a661215773ece946535676/tools/licenses.py [modify] https://crrev.com/35383539aa83f27185a661215773ece946535676/tools/mb/mb.py [modify] https://crrev.com/35383539aa83f27185a661215773ece946535676/tools/mb/mb_unittest.py [modify] https://crrev.com/35383539aa83f27185a661215773ece946535676/tools/traffic_annotation/auditor/traffic_annotation_auditor.cc
,
Jun 28 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fed0e8ec393c317037a824b63b3e9690b8b0a4fc commit fed0e8ec393c317037a824b63b3e9690b8b0a4fc Author: Scott Graham <scottmg@chromium.org> Date: Thu Jun 28 05:02:14 2018 Pull GN via CIPD package This is a reland of https://chromium-review.googlesource.com/c/chromium/src/+/1112840. The gn binary will be downloaded into third_party/gn. This CL rolls buildtools forward to a revision where the gn binary is a stub that prints an error message and returns 1. This is to avoid depending on the old location, and helping users relying on gn auto regen. Update mb.py to use the new location. Update explicit location in ios/web_view/BUILD.gn for the new location. Update explicit location in tools/traffic_annotation/auditor/traffic_annotation_auditor.cc for the new location. Update explicit location in tools/licenses.py with for the new location. Update explicit location in testing/libfuzzer/gen_fuzzer_owners.py. Bug: 855791 Bug: 856883 Bug: 856884 Bug: 856878 Bug: 856899 Bug: 857107 Bug: 857110 Change-Id: Iedaa29161e56cc90beafd802efca3a6c12859c58 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Reviewed-on: https://chromium-review.googlesource.com/1117264 Commit-Queue: Scott Graham <scottmg@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#571031} [modify] https://crrev.com/fed0e8ec393c317037a824b63b3e9690b8b0a4fc/DEPS [modify] https://crrev.com/fed0e8ec393c317037a824b63b3e9690b8b0a4fc/ios/web_view/BUILD.gn [modify] https://crrev.com/fed0e8ec393c317037a824b63b3e9690b8b0a4fc/testing/libfuzzer/gen_fuzzer_owners.py [modify] https://crrev.com/fed0e8ec393c317037a824b63b3e9690b8b0a4fc/tools/licenses.py [modify] https://crrev.com/fed0e8ec393c317037a824b63b3e9690b8b0a4fc/tools/mb/mb.py [modify] https://crrev.com/fed0e8ec393c317037a824b63b3e9690b8b0a4fc/tools/mb/mb_unittest.py [modify] https://crrev.com/fed0e8ec393c317037a824b63b3e9690b8b0a4fc/tools/traffic_annotation/auditor/traffic_annotation_auditor.cc
,
Jun 28 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8962649114490c145c5cba76fe290752014d6432 commit 8962649114490c145c5cba76fe290752014d6432 Author: Xiaoqian Dai <xdai@chromium.org> Date: Thu Jun 28 17:07:46 2018 Revert "Pull GN via CIPD package" This reverts commit fed0e8ec393c317037a824b63b3e9690b8b0a4fc. Reason for revert: It broke most of the chrome_informational builders at BuildPackages stage. See crbug.com/857133 for details. Original change's description: > Pull GN via CIPD package > > This is a reland of > https://chromium-review.googlesource.com/c/chromium/src/+/1112840. > > > The gn binary will be downloaded into third_party/gn. This CL rolls > buildtools forward to a revision where the gn binary is a stub that > prints an error message and returns 1. This is to avoid depending > on the old location, and helping users relying on gn auto regen. > > Update mb.py to use the new location. > > Update explicit location in ios/web_view/BUILD.gn for the new location. > > Update explicit location in > tools/traffic_annotation/auditor/traffic_annotation_auditor.cc for the new > location. > > Update explicit location in tools/licenses.py with for the new location. > > Update explicit location in testing/libfuzzer/gen_fuzzer_owners.py. > > Bug: 855791 > Bug: 856883 > Bug: 856884 > Bug: 856878 > Bug: 856899 > Bug: 857107 > Bug: 857110 > Change-Id: Iedaa29161e56cc90beafd802efca3a6c12859c58 > Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet > Reviewed-on: https://chromium-review.googlesource.com/1117264 > Commit-Queue: Scott Graham <scottmg@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Cr-Commit-Position: refs/heads/master@{#571031} TBR=dpranke@chromium.org,scottmg@chromium.org Change-Id: I28962c879b7bc24e809b29427122bb425e9450a5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 855791, 856883, 856884 , 856878, 856899 , 857107, 857110 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Reviewed-on: https://chromium-review.googlesource.com/1118825 Reviewed-by: Xiaoqian Dai <xdai@chromium.org> Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#571170} [modify] https://crrev.com/8962649114490c145c5cba76fe290752014d6432/DEPS [modify] https://crrev.com/8962649114490c145c5cba76fe290752014d6432/ios/web_view/BUILD.gn [modify] https://crrev.com/8962649114490c145c5cba76fe290752014d6432/testing/libfuzzer/gen_fuzzer_owners.py [modify] https://crrev.com/8962649114490c145c5cba76fe290752014d6432/tools/licenses.py [modify] https://crrev.com/8962649114490c145c5cba76fe290752014d6432/tools/mb/mb.py [modify] https://crrev.com/8962649114490c145c5cba76fe290752014d6432/tools/mb/mb_unittest.py [modify] https://crrev.com/8962649114490c145c5cba76fe290752014d6432/tools/traffic_annotation/auditor/traffic_annotation_auditor.cc |
||
►
Sign in to add a comment |
||
Comment 1 by scottmg@chromium.org
, Jun 27 2018Status: Started (was: Untriaged)