diff options
author | Markus Lehtonen <markus.lehtonen@linux.intel.com> | 2014-05-13 16:41:32 +0300 |
---|---|---|
committer | Guido Günther <agx@sigxcpu.org> | 2015-04-01 20:46:40 +0200 |
commit | 1e6672bfda9131eedb6adf158cddd19091e1d8c7 (patch) | |
tree | 97b3056bc0474f100e8f45d56dbc4a7bfc65666b | |
parent | 53fbee6108018ba49b50795d539dfdbe19d04298 (diff) |
notifications: make notify() more generic
In order to be able to re-use it in gbp-buildpackage-rpm.
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
-rw-r--r-- | gbp/notifications.py | 5 | ||||
-rwxr-xr-x | gbp/scripts/buildpackage.py | 11 |
2 files changed, 8 insertions, 8 deletions
diff --git a/gbp/notifications.py b/gbp/notifications.py index 9d9071a9..d628b269 100644 --- a/gbp/notifications.py +++ b/gbp/notifications.py @@ -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) diff --git a/gbp/scripts/buildpackage.py b/gbp/scripts/buildpackage.py index 04971594..4aac21b5 100755 --- a/gbp/scripts/buildpackage.py +++ b/gbp/scripts/buildpackage.py @@ -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 |