Re: [Spice-devel] [PATCH spice-gtk 1/3] Rename snappy to spice-screenshot

2013-04-10 Thread Christophe Fergeau
On Wed, Apr 10, 2013 at 04:50:18PM +0200, Hans de Goede wrote: > >spicy-screenshot (also in Subject). > > Erm I like spice-screenshot better TBH. This goes well with gnome-screenshot Christophe pgpGN4S7DCEKB.pgp Description: PGP signature ___ Spice-d

Re: [Spice-devel] [PATCH spice-gtk 1/3] Rename snappy to spice-screenshot

2013-04-10 Thread Hans de Goede
Hi, On 04/10/2013 03:05 PM, Uri Lublin wrote: Hi Marc-Andre, Looks good, with two comments: On 04/09/2013 10:02 PM, Marc-André Lureau wrote: Following discussion on the ML: http://lists.freedesktop.org/archives/spice-devel/2013-April/012953.html --- README | 2 +- gtk/Makefile.

Re: [Spice-devel] [PATCH spice-gtk 1/3] Rename snappy to spice-screenshot

2013-04-10 Thread Uri Lublin
Hi Marc-Andre, Looks good, with two comments: On 04/09/2013 10:02 PM, Marc-André Lureau wrote: Following discussion on the ML: http://lists.freedesktop.org/archives/spice-devel/2013-April/012953.html --- README | 2 +- gtk/Makefile.am | 12 ++-- gtk/snappy.c| 178 --

Re: [Spice-devel] [PATCH spice-gtk 1/3] Rename snappy to spice-screenshot

2013-04-09 Thread Hans de Goede
Hi, ACK series, but before release the spice-common patch titled: "[spice-common PATCH] supply missing IS_IMAGE_TYPE_* elements for LZ_IMAGE_TYPE_A8" Should be pushed to spice-common and the spice-common sub-module of spice-gtk should be updated. Regards, Hans On 04/09/2013 09:02 PM, Marc-And

[Spice-devel] [PATCH spice-gtk 1/3] Rename snappy to spice-screenshot

2013-04-09 Thread Marc-André Lureau
Following discussion on the ML: http://lists.freedesktop.org/archives/spice-devel/2013-April/012953.html --- README | 2 +- gtk/Makefile.am | 12 ++-- gtk/snappy.c| 178 po/POTFILES.in | 2 +- 4 files changed, 8 insertions