DescriptionFix various issues in the roll_gn script.
This patch address a bunch of issues people have found in the roll_gn script:
- Fixes yet another bug where we weren't waiting for the try jobs
to finish.
- Fixes a bug where we were adding an extra '\n' onto the DEPS file
in the final roll CL.
- Closes the 'build_gn' CL once the build has completed.
- Adds better logging at the end of the 'wait' and 'roll_buildtools' steps.
- Removes mac_chromium_gn_rel from CQ_EXTRA_TRYBOTS, since
it is part of the default set now.
R=brettw@chromium.org, andybons@chromium.org
BUG=
Committed: https://crrev.com/513d94882c9720688a3e3f67da2b0a529e27b7b9
Cr-Commit-Position: refs/heads/master@{#351141}
Patch Set 1 #Patch Set 2 : remove mac_chromium_gn_rel from CQ_EXTRA_TRYBOTS #Messages
Total messages: 7 (1 generated)
|