More bug fixes for multi-distro support

[invirt/packages/invirt-dev.git] / build-hooks / post-build

diff --git a/build-hooks/post-build b/build-hooks/post-build
index 8dd8eb4..3b9d3f4 100755 (executable)
--- a/build-hooks/post-build
+++ b/build-hooks/post-build
@@ -96,27 +96,41 @@ def submit_completion_msg(succeeded, values, verbose=True, success=lambda x: x,
     values = dict(values)
     if succeeded:
         values['result'] = success(values['result'])
+        values['_extra'] = ''
     else:
         values['result'] = failure(values['result'])
+        values['_extra'] = '\nError: %s' % escape(sys.stdin.read())
     for key in ['commit', 'pocket', 'principal']:
         values[key] = escape(values[key])
     msg = """Submission of %(commit)s to be built in %(pocket)s %(result)s.
-Build submitted by %(principal)s.""" % values
+Build submitted by %(principal)s.%(_extra)s""" % values
     return msg
 
 def repo_creation_msg(succeeded, values, verbose=True, success=lambda x: x, failure=lambda x: x,
                       escape=lambda x: x):
     values = dict(values)
     assert succeeded
+    for key in ['category', 'name', 'principal']:
+        values[key] = escape(values[key])
     msg = '%(principal)s just created a new repository, %(category)s/%(name)s.git' % values
     return msg
 
+def prebuild_msg(succeeded, values, verbose=True, success=lambda x: x, failure=lambda x: x,
+                 escape=lambda x: x):
+    for key in ['build_id', 'pocket', 'package', 'commit', 'principal', 'version', 'inserted_at']:
+        values[key] = escape(values[key])
+    msg = """Build started: %(package)s %(version)s in %(pocket)s.
+Base commit %(commit)s .
+Job submitted by %(principal)s at %(inserted_at)s.""" % values
+    return msg
+
 # Names of hooks
 POST_BUILD = 'post-build'
 FAILED_BUILD = 'failed-build'
 POST_SUBMIT = 'post-submit'
 FAILED_SUBMIT = 'failed-submit'
 POST_ADD_REPO = 'post-add-repo'
+PRE_BUILD = 'pre-build'
 
 # Types of communication
 
@@ -128,12 +142,14 @@ message_generators = {
                FAILED_BUILD : build_completion_msg,
                POST_SUBMIT : submit_completion_msg,
                FAILED_SUBMIT : submit_completion_msg,
-               POST_ADD_REPO : repo_creation_msg },
+               POST_ADD_REPO : repo_creation_msg,
+               PRE_BUILD : prebuild_msg },
     MAIL   : { POST_BUILD : build_completion_msg,
                FAILED_BUILD : build_completion_msg,
                POST_SUBMIT : submit_completion_msg,
                FAILED_SUBMIT : submit_completion_msg,
-               POST_ADD_REPO : repo_creation_msg }
+               POST_ADD_REPO : repo_creation_msg,
+               PRE_BUILD : prebuild_msg }
     }
 
 def zephyr_escape(m):
@@ -164,6 +180,8 @@ def main():
             hook_config = config.build.hooks.failed_submit
         elif prog == POST_ADD_REPO:
             hook_config = config.build.hooks.post_add_repo
+        elif prog == PRE_BUILD:
+            hook_config = config.build.hooks.pre_build
         else:
             parser.error('hook script invoked with unrecognized name %s' % prog)
             return 2
@@ -171,7 +189,7 @@ def main():
         print >>sys.stderr, 'No hook configuration found for %s.' % prog
         return 1
 
-    if prog in [POST_BUILD, FAILED_BUILD]:
+    if prog in [POST_BUILD, FAILED_BUILD, PRE_BUILD]:
         if len(args) != 1:
             parser.set_usage('Usage: %prog [options] build_id')
             parser.print_help()
@@ -191,7 +209,9 @@ def main():
                    'version' : build.version,
                    'default_instance' : 'b%(build_id)s',
                    'default_subject' : 'Build %(build_id)d %(result)s: %(package)s %(version)s in %(pocket)s'}
-        if build.succeeded:
+        if prog == PRE_BUILD:
+            succeeded = True
+        elif build.succeeded:
             assert prog == POST_BUILD
             values['result'] = 'succeeded'
             succeeded = True