Bug 40889 - Sheriffbot: Should allow "r" in SVN_REVISION
Summary: Sheriffbot: Should allow "r" in SVN_REVISION
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-20 12:16 PDT by Joseph Pecoraro
Modified: 2010-06-20 15:51 PDT (History)
6 users (show)

See Also:


Attachments
Strip a Leading "r" if its there. (2.46 KB, patch)
2010-06-20 12:19 PDT, Joseph Pecoraro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2010-06-20 12:16:06 PDT
Situation I just had:

  JoePeck:    sheriffbot: rollout r61506 Causes crash. Will take a different approach.
  sheriffbot: Preparing rollout for rr61506...
  sheriffbot: JoePeck: Failed to create rollout patch:
  sheriffbot: Invalid svn revision number "r61506".

=(. Bot could be more lenient.
Comment 1 Joseph Pecoraro 2010-06-20 12:19:20 PDT
Created attachment 59209 [details]
Strip a Leading "r" if its there.

NOTE: I didn't run the tests! How do I run the python
tests for the irc bot?
Comment 2 Joseph Pecoraro 2010-06-20 12:23:10 PDT
CC'ing a bunch of people who have worked in the WebKitTools/Scripts/webkitpy/tool
directory. The patch is simple enough, but see my question above about running the
tests for the tools. Thanks!
Comment 3 Adam Barth 2010-06-20 12:34:43 PDT
You can run test-webkitpy
Comment 4 Adam Barth 2010-06-20 12:41:47 PDT
Comment on attachment 59209 [details]
Strip a Leading "r" if its there.

Lgtm
Comment 5 Joseph Pecoraro 2010-06-20 12:49:11 PDT
(In reply to comment #3)
> You can run test-webkitpy

Excellent, that worked perfectly!

Has it been considered that this be renamed to match the usual
run-*-tests pattern? I see there are a few like this, all for scripts:

  WebKitTools/Scripts/test-webkit-scripts
  WebKitTools/Scripts/test-webkitperl
  WebKitTools/Scripts/test-webkitpy
Comment 6 Adam Barth 2010-06-20 13:04:51 PDT
You can look at the discussion on the bug where thoses were added.  I think various folks preferred the test-* pattern for reasons I don't remember.
Comment 7 Joseph Pecoraro 2010-06-20 13:08:54 PDT
(In reply to comment #6)
> You can look at the discussion on the bug where thoses were added.
> I think various folks preferred the test-* pattern for reasons I don't
> remember.

Ahh, okay. Thanks!
Comment 8 WebKit Commit Bot 2010-06-20 15:51:25 PDT
Comment on attachment 59209 [details]
Strip a Leading "r" if its there.

Clearing flags on attachment: 59209

Committed r61515: <http://trac.webkit.org/changeset/61515>
Comment 9 WebKit Commit Bot 2010-06-20 15:51:30 PDT
All reviewed patches have been landed.  Closing bug.