Commit 1e6672bf authored by Markus Lehtonen's avatar Markus Lehtonen Committed by Guido Günther

notifications: make notify() more generic

In order to be able to re-use it in gbp-buildpackage-rpm.
Signed-off-by: 's avatarMarkus Lehtonen <markus.lehtonen@linux.intel.com>
parent 53fbee61
......@@ -53,7 +53,7 @@ def send_notification(summary, msg):
return True
def notify(cp, success, notify_opt):
def notify(summary, message, notify_opt):
"""
Send a notifications
@return: False on error
......@@ -66,6 +66,5 @@ def notify(cp, success, notify_opt):
if not enable:
return [True, False][notify_opt.is_on()]
summary, msg = build_msg(cp, success)
return notify_opt.do(send_notification, summary, msg)
return notify_opt.do(send_notification, summary, message)
......@@ -636,11 +636,12 @@ def main(argv):
if options.export_dir and options.purge and not retval:
RemoveTree(export_dir)()
if source and not gbp.notifications.notify(source.changelog,
not retval,
options.notify):
gbp.log.err("Failed to send notification")
retval = 1
if source:
summary, msg = gbp.notifications.build_msg(source.changelog,
not retval)
if not gbp.notifications.notify(summary, msg, options.notify):
gbp.log.err("Failed to send notification")
retval = 1
return retval
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment