iOS build looking in buildtools/mac/gn explicitly somehow |
||||
Issue descriptionninja: Entering directory `/b/s/w/ir/cache/builder/src/out/Debug-iphonesimulator' ninja: error: '../../buildtools/mac/gn', needed by 'ios_web_view/LICENSE', missing and no known rule to make it step returned non-zero exit code: 1 https://chromium-review.googlesource.com/c/chromium/src/+/1112840 https://ci.chromium.org/p/chromium/builders/luci.chromium.try/ios-simulator/37706 https://cs.chromium.org/chromium/src/ios/web_view/BUILD.gn?type=cs&q=buildtools.*gn&sq=package:chromium&g=0&l=432
,
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
(The ref in c#0 is fixed, CL out for c#1, but I'm still trying to figure out how to update expectations in build_limited so I'll leave this open for now.)
,
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/tools/build/+/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9 commit 13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9 Author: Scott Graham <scottmg@chromium.org> Date: Thu Jun 28 05:02:54 2018 Update scripts/slave/recipe_modules/ios/api.py for new gn location Bug: 856883 Change-Id: I20d3d8138721af59b108401dbe419de016237f5d Recipe-Nontrivial-Roll: build_limited_scripts_slave Reviewed-on: https://chromium-review.googlesource.com/1116287 Reviewed-by: Dirk Pranke <dpranke@chromium.org> Commit-Queue: Scott Graham <scottmg@chromium.org> [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/perf_test.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/infra_failure.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/explain.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/xcode_build_version.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/timed_out.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/api.py [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/basic.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/no_exit_code.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/device_check_false.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/deprecate_xcode_version.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/errors.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/expiration_test.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/clobber_build.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/fyi.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/expired.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/xcode_build_version_luci.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/test_failure.json [modify] https://crrev.com/13fe2e7b4e6e876f0e4ce63508a0ce88338b9ad9/scripts/slave/recipe_modules/ios/examples/full.expected/clobber_checkout.json
,
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
,
Jul 2
,
Jul 9
Is the issue in c #0 still fixed after the revert? If not, we can probably mimic cronet's generate license script which doesn't explicitly use gn as an input: https://cs.chromium.org/chromium/src/components/cronet/ios/BUILD.gn?rcl=7f1ae6a37682b2e6cdd3de89f95f29ae1f357af8&l=287
,
Jul 9
|
||||
►
Sign in to add a comment |
||||
Comment 1 by scottmg@chromium.org
, Jun 27 2018