Author: David Schneider <david.schnei...@picle.org>
Branch: buildbot-0.8.7
Changeset: r805:b48b1aac38d1
Date: 2013-04-26 21:04 +0200
http://bitbucket.org/pypy/buildbot/changeset/b48b1aac38d1/

Log:    raise a validation error instead of crashing if the username is
        missing

diff --git a/bot2/pypybuildbot/master.py b/bot2/pypybuildbot/master.py
--- a/bot2/pypybuildbot/master.py
+++ b/bot2/pypybuildbot/master.py
@@ -2,6 +2,7 @@
 import os
 from buildbot.scheduler import Nightly
 from buildbot.schedulers.forcesched import ForceScheduler
+from buildbot.schedulers.forcesched import ValidationError
 from buildbot.buildslave import BuildSlave
 from buildbot.status.html import WebStatus
 #from buildbot import manhole
@@ -10,10 +11,10 @@
 from pypybuildbot.util import we_are_debugging
 
 # Forbid "force build" with empty user name
-
 class CustomForceScheduler(ForceScheduler):
     def force(self, owner, builder_name, **kwargs):
-        assert owner, "Please write your name in the corresponding field."
+        if not owner:
+            raise ValidationError, "Please write your name in the 
corresponding field."
         return ForceScheduler.force(self, owner, builder_name, **kwargs)
 
 
_______________________________________________
pypy-commit mailing list
pypy-commit@python.org
http://mail.python.org/mailman/listinfo/pypy-commit

Reply via email to