[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