|
|
Created:
4 years, 3 months ago by Dirk Pranke Modified:
4 years, 3 months ago Reviewers:
brettw CC:
chromium-reviews Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
DescriptionRe-land "Update MB to use `gn analyze`."
This change re-lands r411530, with a bunch of additional targets
added to gn_isolate_map.pyl to handle all of the other compile and
test targets (including mappings for apk's and junit tests).
TBR=brettw@chromium.org
BUG=555273
Committed: https://crrev.com/cb4a2e24a10c9063718e266d94738843898a7c60
Cr-Commit-Position: refs/heads/master@{#419404}
Patch Set 1 : original patch (the version that was reverted) #Patch Set 2 : add changes for testing purposes #Patch Set 3 : add missing targets #Patch Set 4 : remove testing changes #
Messages
Total messages: 36 (33 generated)
The CQ bit was checked by dpranke@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...) linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
The CQ bit was checked by dpranke@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...) linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by dpranke@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
The CQ bit was checked by dpranke@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
The CQ bit was checked by dpranke@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
Patchset #3 (id:40001) has been deleted
Patchset #3 (id:60001) has been deleted
Patchset #3 (id:80001) has been deleted
Patchset #3 (id:90001) has been deleted
Description was changed from ========== Re-land "Update MB to use `gn analyze`." BUG= ========== to ========== Re-land "Update MB to use `gn analyze`." This change re-lands r411530, with a bunch of additional targets added to gn_isolate_map.pyl to handle all of the other compile and test targets (including mappings for apk's and junit tests). TBR=brettw@chromium.org BUG=555273 ==========
dpranke@chromium.org changed reviewers: + brettw@chromium.org
The CQ bit was checked by dpranke@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by dpranke@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Message was sent while issue was closed.
Description was changed from ========== Re-land "Update MB to use `gn analyze`." This change re-lands r411530, with a bunch of additional targets added to gn_isolate_map.pyl to handle all of the other compile and test targets (including mappings for apk's and junit tests). TBR=brettw@chromium.org BUG=555273 ========== to ========== Re-land "Update MB to use `gn analyze`." This change re-lands r411530, with a bunch of additional targets added to gn_isolate_map.pyl to handle all of the other compile and test targets (including mappings for apk's and junit tests). TBR=brettw@chromium.org BUG=555273 ==========
Message was sent while issue was closed.
Committed patchset #4 (id:130001)
Message was sent while issue was closed.
Description was changed from ========== Re-land "Update MB to use `gn analyze`." This change re-lands r411530, with a bunch of additional targets added to gn_isolate_map.pyl to handle all of the other compile and test targets (including mappings for apk's and junit tests). TBR=brettw@chromium.org BUG=555273 ========== to ========== Re-land "Update MB to use `gn analyze`." This change re-lands r411530, with a bunch of additional targets added to gn_isolate_map.pyl to handle all of the other compile and test targets (including mappings for apk's and junit tests). TBR=brettw@chromium.org BUG=555273 Committed: https://crrev.com/cb4a2e24a10c9063718e266d94738843898a7c60 Cr-Commit-Position: refs/heads/master@{#419404} ==========
Message was sent while issue was closed.
Patchset 4 (id:??) landed as https://crrev.com/cb4a2e24a10c9063718e266d94738843898a7c60 Cr-Commit-Position: refs/heads/master@{#419404} |