The link to the rollout is suffixed with ':' which makes it invalid URL. e.g. Vsevolod Vlasov (vsevik) rolled out r144016 in https://trac.webkit.org/r144021: Breaks compilation on chromium mac Removing the ':' from the WKR message. Patch follows.
Created attachment 190239 [details] Patch
Comment on attachment 190239 [details] Patch Sounds like you have a bug in your IRC client. : is a reserved word in URL. It shouldn't be treated like a part of URL.
Oops, apparently colon is allowed to appear in paths of an URL: http://tools.ietf.org/html/rfc3986
Comment on attachment 190239 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=190239&action=review > Tools/Scripts/webkitpy/tool/commands/newcommitbot.py:150 > - return '%s rolled out %s in %s: %s' % (author, rollout.group('revisions'), > + return '%s rolled out %s in %s %s' % (author, rollout.group('revisions'), Instead of removing the colon, please insert a space before it.
Created attachment 190242 [details] Patch
Comment on attachment 190242 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=190242&action=review > Tools/ChangeLog:3 > + [Tools - WKR]Fix the links shown for rollouts by WKR I've updated the bug title to be more describe. Please update the change log accordingly.
Created attachment 190243 [details] Patch
Comment on attachment 190243 [details] Patch Clearing flags on attachment: 190243 Committed r144040: <http://trac.webkit.org/changeset/144040>
All reviewed patches have been landed. Closing bug.
This has caused a regression in the Python tests. See <http://build.webkit.org/builders/EFL%20Linux%2064-bit%20Release%20WK2/builds/5360/steps/webkitpy-test/logs/stdio>.
(In reply to comment #10) > This has caused a regression in the Python tests. See <http://build.webkit.org/builders/EFL%20Linux%2064-bit%20Release%20WK2/builds/5360/steps/webkitpy-test/logs/stdio>. https://bugs.webkit.org/attachment.cgi?id=190299&action=review should fix this failure. Thank you.