Improve the error handling in rollout a bit
Created attachment 52360 [details] Patch
Comment on attachment 52360 [details] Patch dave_levin broke my patch. Will re-spin.
Created attachment 52365 [details] Patch
Comment on attachment 52365 [details] Patch Explain why in your comment, please: # Note: Don't print a bug URL here because it will confuse the 259 # SheriffBot. Yes, that's lame. 260 log("Preparing rollout for bug %s." % commit_info.bug_id()) 261 cc_list? 267 cc = sorted([party.bugzilla_email() We should be passing around lists instead of a string: 273 "bug_cc": ",".join(cc), Anyway, this looks fine. We should fix the cc passing around at some point. Ideally now, but not required. Either way I don't need to see this again.
Created attachment 52370 [details] Patch for landing
Comment on attachment 52370 [details] Patch for landing Clearing flags on attachment: 52370 Committed r56964: <http://trac.webkit.org/changeset/56964>
All reviewed patches have been landed. Closing bug.