Bug 37001

Summary:

sheriffbot rollout should verify that the svn revision is a number.

Product:

WebKit

Reporter:

David Levin <levin>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, eric

Priority:

Version:

528+ (Nightly build)

Hardware:

Other

OS:

OS X 10.5

Attachments:

Description	Flags
Patch	abarth: review+

Description David Levin 2010-04-01 18:33:16 PDT

sheriffbot rollout should verify that the svn revision is a number.

Comment 1 David Levin 2010-04-01 18:37:03 PDT

Created attachment 52364 [details]
Patch

Comment 2 Adam Barth 2010-04-01 18:41:03 PDT

Comment on attachment 52364 [details]
Patch

This is good, but I'm about to remove the IRC post of the ScriptError.  You should add it back when you land this though.

Also, we should validate that the first character of reason isn't a "-".

Comment 3 David Levin 2010-04-01 18:45:56 PDT

Committed as http://trac.webkit.org/changeset/56960