Summary: | Pause during ANGLE roll script after rebasing and before deleting temporary Git repo | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kenneth Russell <kbr> | ||||
Component: | ANGLE | Assignee: | Kenneth Russell <kbr> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | dino, ews-watchlist, graouts, jdarpinian, justin_fan, kondapallykalyan, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 214897 | ||||||
Attachments: |
|
Description
Kenneth Russell
2020-06-03 15:13:37 PDT
Taking this. Created attachment 405418 [details]
Patch
Note that there are important steps to take when updating ANGLE. See http://trac.webkit.org/wiki/UpdatingANGLE This patch also slightly changes the user interface of the roll script to expect the user to press enter after confirming (y/n) whether to continue. I found it surprising on multiple occasions that the script went ahead immediately upon pressing the "y" key. Looks good to me! I was thinking that the script would exit to let you poke around but this is much easier to implement and I think just as useful. Thanks for your review James. I also thought of exiting the script but this was a lot easier to implement since it doesn't need any state management. It turned out to be useful during the ANGLE roll I'm currently doing in Bug 214897 - it caught one bad merge during rebase, and another important fix to upstream to ANGLE (Bug 213479, http://crbug.com/angleproject/4902 ). Committed r265023: <https://trac.webkit.org/changeset/265023> All reviewed patches have been landed. Closing bug and clearing flags on attachment 405418 [details]. |