Ticket #138: 0010-Allow-patches-to-be-submitted-through-the-web-interf.patch

buildbot/sourcestamp.py

@@ -12 +12 @@
        None, build the HEAD revision from the given branch.
      - (revision=REV, patchspec=(LEVEL, DIFF), changes=None): checkout REV,
        then apply a patch to the source, with C{patch -pPATCHLEVEL <DIFF}.
-       If REV is None, checkout HEAD and patch it.
+       If REV is C{None}, checkout HEAD and patch it.  DIFF is a string that
+       contains the unified diff to apply.
      - (revision=None, patchspec=None, changes=[CHANGES]): let the Source
        step check out the latest revision indicated by the given Changes.
        CHANGES is a tuple of L{buildbot.changes.changes.Change} instances,

buildbot/status/web/base.py

@@ -95 +95 @@
         where = " on all builders"
     else:
         where = ""
-    data = """<form action="%s" class="command forcebuild">
+    if useLoginPassword:
+        form_type = ' method="post" enctype="multipart/form-data"'
+        form_attach = (make_row("Build with a patch?",
+                                '<input type="file" name="patch" />')
+                       + make_row("Patch level:",
+                                  '<input type="text" name="plevel" size="2" '
+                                  'maxlength="2" value="1" />'))
+        form_attach = ""
+        form_type = ""
+
+    data = """<form action="%s"%s class="command forcebuild">
       <p>To force a build%s, fill out the following fields and
-      click the `Force Build' button</p>""" % (forceURL, where)
+      click the `Force Build' button</p>
+    """ % (forceURL, form_type, where)
     return (data
       + make_name_login_password_form(useLoginPassword)
       + make_row("Reason for build:",
@@ -106 +117 @@
                  '<input type="text" name="branch" />')
       + make_row("Revision to build:",
                  '<input type="text" name="revision" />')
+      + form_attach
       + '<input type="submit" value="Force Build" /></form>\n')
 
 colormap = {

buildbot/status/web/builder.py

@@ -132 +132 @@
         reason = req.args.get("comments", ["<no reason specified>"])[0]
         branch = req.args.get("branch", [""])[0]
         revision = req.args.get("revision", [""])[0]
+        patch = req.args.get("patch", [""])[0]
+        if patch:
+            patch_level = req.args.get("plevel", ["1"])[0]
+            try:
+                patch_level = int(patch_level)
+            except ValueError:
+                # TODO: tell the web user that their request was invalid
+                return Redirect("..")
+            patch = (patch_level, patch)
+            has_patch = "yes"
+        else:
+            has_patch = "no"
+            patch = None
 
-        r = "The web-page 'force build' button was pressed by '%s': %s\n" \
-            % (name, reason)
-        log.msg("web forcebuild of builder '%s', branch='%s', revision='%s'"
-                "by user '%s'" % (self.builder_status.getName(), branch,
-                                  revision, name))
+        r = "The web-page 'force build' button was pressed by '%s': %s" \
+            "(patch: %s)\n" % (name, reason, has_patch)
+        log.msg("web forcebuild of builder '%s', branch='%s', "
+                "revision='%s', by user '%s' (patch: %s)" \
+                % (self.builder_status.getName(), branch, revision, name,
+                   has_patch))
 
         if not self.builder_control:
             # TODO: tell the web user that their request was denied
@@ -148 +162 @@
             if not self.authUser(req):
                 # TODO: tell the web user that their request was denied
                 return Redirect("..")
+        else:
+            if patch:
+                log.msg("user is not authenticated: discarding the patch")
+            patch = None
 
         # keep weird stuff out of the branch and revision strings. TODO:
         # centralize this somewhere.
@@ -168 +186 @@
         # buildbot.changes.changes.Change instance which doesn't really fit
         # this use case (it requires a list of changed files which is tedious
         # to compute at this stage)
-        s = SourceStamp(branch=branch, revision=revision)
+        s = SourceStamp(branch=branch, revision=revision, patch=patch)
         req = BuildRequest(r, s, self.builder_status.getName())
         try:
             self.builder_control.requestBuildSoon(req)