[Merge] lp:~seb128/update-notifier/work-under-wayland into lp:update-notifier
Brian Murray
brian at ubuntu.com
Wed Sep 6 16:59:31 UTC 2017
Review: Needs Fixing
This looks good to me, aside of one in-line comment, thanks!
Diff comments:
> === modified file 'debian/changelog'
> --- debian/changelog 2017-09-05 21:48:13 +0000
> +++ debian/changelog 2017-09-06 09:34:16 +0000
> @@ -1,3 +1,11 @@
> +update-notifier (3.186) UNRELEASED; urgency=medium
> +
> + * src/clipboard.c, src/update-notifier.c, src/Makefile.am:
> + - don't use the xorg clipboard to do single instance, that doesn't work
> + under wayland, replace it with a file lock (lp: #1697381)
I noticed this merge proposal isn't linked to the bug report, I think the "lp:" part needs to be "LP:".
> +
> + -- Sebastien Bacher <seb128 at ubuntu.com> Wed, 06 Sep 2017 11:29:31 +0200
> +
> update-notifier (3.185) artful; urgency=medium
>
> * src/livepatch.c: Use ngettext to correctly handle the plural form, thanks
--
https://code.launchpad.net/~seb128/update-notifier/work-under-wayland/+merge/330271
Your team Ubuntu Core Development Team is requested to review the proposed merge of lp:~seb128/update-notifier/work-under-wayland into lp:update-notifier.
More information about the Ubuntu-reviews
mailing list