rpms/spice-gtk
5
0
Fork 0
Code Issues Pull Requests Releases Activity

Update to spice-gtk 0.21

Browse Source
This commit is contained in:
Marc-André Lureau 2013-09-18 18:47:43 +02:00
parent 8dc5ba9f89
commit ad9d1fabb8
10 changed files with 11 additions and 469 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.gitignore vendored
View File

@ -20,3 +20,4 @@
/spice-gtk-0.18.tar.bz2
/spice-gtk-0.19.tar.bz2
/spice-gtk-0.20.tar.bz2
/spice-gtk-0.21.tar.bz2

91
0001-Add-spice_channel_string_to_type-to-map-files.patch
View File

@ -1,91 +0,0 @@
From b85ca792af72d5f1e10aa6af9515eb96a044fc5c Mon Sep 17 00:00:00 2001
From: Hans de Goede <hdegoede@redhat.com>
Date: Wed, 26 Jun 2013 17:39:39 +0200
Subject: [PATCH spice-gtk] Add spice_channel_string_to_type to map files
And document both spice_channel_string_to_type and
spice_channel_type_to_string.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
doc/reference/spice-gtk-sections.txt | 1 +
gtk/map-file | 1 +
gtk/spice-channel.c | 18 ++++++++++++++++++
gtk/spice-glib-sym-file | 1 +
4 files changed, 21 insertions(+)
diff --git a/doc/reference/spice-gtk-sections.txt b/doc/reference/spice-gtk-sections.txt
index e1e2833..8d61aa9 100644
--- a/doc/reference/spice-gtk-sections.txt
+++ b/doc/reference/spice-gtk-sections.txt
@@ -98,6 +98,7 @@ spice_channel_disconnect
spice_channel_test_capability
spice_channel_test_common_capability
spice_channel_type_to_string
+spice_channel_string_to_type
spice_channel_set_capability
spice_channel_flush_async
spice_channel_flush_finish
diff --git a/gtk/map-file b/gtk/map-file
index a0b7330..a69eb40 100644
--- a/gtk/map-file
+++ b/gtk/map-file
@@ -11,6 +11,7 @@ spice_channel_get_type;
spice_channel_new;
spice_channel_open_fd;
spice_channel_set_capability;
+spice_channel_string_to_type;
spice_channel_test_capability;
spice_channel_test_common_capability;
spice_channel_type_to_string;
diff --git a/gtk/spice-channel.c b/gtk/spice-channel.c
index 0a32d6c..093b292 100644
--- a/gtk/spice-channel.c
+++ b/gtk/spice-channel.c
@@ -1884,6 +1884,15 @@ static const char *to_string[] = {
[ SPICE_CHANNEL_PORT ] = "port",
};
+/**
+ * spice_channel_type_to_string:
+ * @type: a channel-type property value
+ *
+ * Convert a channel-type property value to a string.
+ *
+ * Returns: string representation of @type.
+ * Since: 0.20
+ **/
const gchar* spice_channel_type_to_string(gint type)
{
const char *str = NULL;
@@ -1895,6 +1904,15 @@ const gchar* spice_channel_type_to_string(gint type)
return str ? str : "unknown channel type";
}
+/**
+ * spice_channel_string_to_type:
+ * @str: a string representation of the channel-type property
+ *
+ * Convert a channel-type property value to a string.
+ *
+ * Returns: the channel-type property value for a @str channel
+ * Since: 0.21
+ **/
gint spice_channel_string_to_type(const gchar *str)
{
int i;
diff --git a/gtk/spice-glib-sym-file b/gtk/spice-glib-sym-file
index 2b172d0..8540307 100644
--- a/gtk/spice-glib-sym-file
+++ b/gtk/spice-glib-sym-file
@@ -14,6 +14,7 @@ spice_channel_set_capability
spice_channel_test_capability
spice_channel_test_common_capability
spice_channel_type_to_string
+spice_channel_string_to_type
spice_client_error_quark
spice_cursor_channel_get_type
spice_display_channel_get_type
--
1.8.3.rc1.49.g8d97506

45
0002-smartcard-Handle-VCARD_EMUL_INIT_ALREADY_INITED.patch
View File

@ -1,45 +0,0 @@
From 3bb15dd43daaed6b1d77e66c59432f228935322e Mon Sep 17 00:00:00 2001
From: Christophe Fergeau <cfergeau@redhat.com>
Date: Thu, 11 Jul 2013 15:18:33 +0200
Subject: [spice-gtk] smartcard: Handle VCARD_EMUL_INIT_ALREADY_INITED
When initializing a software smartcard, vcard_emul_init() can
report success, error, or indicate that initialization has already
been done. In this last case, we would assume that an error occurred
instead of behaving as if the initialization succeeded.
vcard_emul_init() can end up being called multiple time if the
smartcard channel gets destroyed and recreated during the lifetime
of the application
Fixes rhbz#815639
---
gtk/smartcard-manager.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/gtk/smartcard-manager.c b/gtk/smartcard-manager.c
index 4b1efe2..2a0e397 100644
--- a/gtk/smartcard-manager.c
+++ b/gtk/smartcard-manager.c
@@ -408,6 +408,7 @@ static gboolean smartcard_manager_init(SpiceSession *session,
{
gchar *emul_args = NULL;
VCardEmulOptions *options = NULL;
+ VCardEmulError emul_init_status;
gchar *dbname = NULL;
GStrv certificates = NULL;
gboolean retval = FALSE;
@@ -448,7 +449,9 @@ static gboolean smartcard_manager_init(SpiceSession *session,
init:
SPICE_DEBUG("vcard_emul_init");
- if (vcard_emul_init(options) != VCARD_EMUL_OK) {
+ emul_init_status = vcard_emul_init(options);
+ if ((emul_init_status != VCARD_EMUL_OK)
+ && (emul_init_status != VCARD_EMUL_INIT_ALREADY_INITED)) {
*err = g_error_new(SPICE_CLIENT_ERROR,
SPICE_CLIENT_ERROR_FAILED,
"Failed to initialize smartcard");
--
1.8.3.1

130
0003-channel-main-Convert-text-line-endings-if-necessary-.patch
View File

@ -1,130 +0,0 @@
From 28ef25781f62d6e3d96dba57a7ec772899b83661 Mon Sep 17 00:00:00 2001
From: Hans de Goede <hdegoede@redhat.com>
Date: Mon, 24 Jun 2013 14:30:43 +0200
Subject: [PATCH] channel-main: Convert text line-endings if necessary
(rhbz#752350)
This implements line-ending conversion following the specification in the
commit message of spice-protocol commit 7be0e88e7e03a956b364cc847aad11b96ed4 :
vd_agent: Add caps for the agent to signal the guest line-ending (rhbz#752350)
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
(cherry picked from commit e45a446a9981ad4adaeff9c885962a8c6140333e)
---
gtk/channel-main.c | 73 +++++++++++++++++++++++++++++++++++++++++++++++++++---
1 file changed, 69 insertions(+), 4 deletions(-)
diff --git a/gtk/channel-main.c b/gtk/channel-main.c
index b58af52..b9e0da2 100644
--- a/gtk/channel-main.c
+++ b/gtk/channel-main.c
@@ -1181,6 +1181,24 @@ static void agent_announce_caps(SpiceMainChannel *channel)
#define HAS_CLIPBOARD_SELECTION(c) \
VD_AGENT_HAS_CAPABILITY((c)->agent_caps, G_N_ELEMENTS((c)->agent_caps), VD_AGENT_CAP_CLIPBOARD_SELECTION)
+#define GUEST_LINEEND_LF(c) \
+ VD_AGENT_HAS_CAPABILITY((c)->agent_caps, G_N_ELEMENTS((c)->agent_caps), VD_AGENT_CAP_GUEST_LINEEND_LF)
+
+#define GUEST_LINEEND_CRLF(c) \
+ VD_AGENT_HAS_CAPABILITY((c)->agent_caps, G_N_ELEMENTS((c)->agent_caps), VD_AGENT_CAP_GUEST_LINEEND_CRLF)
+
+#ifdef G_OS_UNIX
+#define CLIENT_LINEEND_LF 1
+#else
+#define CLIENT_LINEEND_LF 0
+#endif
+
+#ifdef G_OS_WIN32
+#define CLIENT_LINEEND_CRLF 1
+#else
+#define CLIENT_LINEEND_CRLF 0
+#endif
+
/* any context: the message is not flushed immediately,
you can wakeup() the channel coroutine or send_msg_queue() */
static void agent_clipboard_grab(SpiceMainChannel *channel, guint selection,
@@ -1751,6 +1769,29 @@ static void file_xfer_handle_status(SpiceMainChannel *channel,
file_xfer_completed(task, error);
}
+/* any context */
+static guchar *convert_lineend(const guchar *in, gsize *size,
+ const gchar *from, const gchar *to)
+{
+ gchar *nul_terminated, **split, *out;
+
+ /* Nul-terminate */
+ nul_terminated = g_malloc(*size + 1);
+ memcpy(nul_terminated, in, *size);
+ nul_terminated[*size] = 0;
+
+ /* Convert */
+ split = g_strsplit(nul_terminated, from, -1);
+ out = g_strjoinv(to, split);
+ *size = strlen(out);
+
+ /* Clean-up */
+ g_strfreev(split);
+ g_free(nul_terminated);
+
+ return (guchar *)out;
+}
+
/* coroutine context */
static void main_agent_handle_msg(SpiceChannel *channel,
VDAgentMessage *msg, gpointer payload)
@@ -1809,12 +1850,22 @@ static void main_agent_handle_msg(SpiceChannel *channel,
case VD_AGENT_CLIPBOARD:
{
VDAgentClipboard *cb = payload;
+ guchar *data = cb->data;
+ gsize size = msg->size - sizeof(VDAgentClipboard);
+ if (cb->type == VD_AGENT_CLIPBOARD_UTF8_TEXT) {
+ if (GUEST_LINEEND_LF(c) && CLIENT_LINEEND_CRLF)
+ data = convert_lineend(data, &size, "\n", "\r\n");
+ if (GUEST_LINEEND_CRLF(c) && CLIENT_LINEEND_LF)
+ data = convert_lineend(data, &size, "\r\n", "\n");
+ }
emit_main_context(channel, SPICE_MAIN_CLIPBOARD_SELECTION, selection,
- cb->type, cb->data, msg->size - sizeof(VDAgentClipboard));
+ cb->type, data, size);
- if (selection == VD_AGENT_CLIPBOARD_SELECTION_CLIPBOARD)
+ if (selection == VD_AGENT_CLIPBOARD_SELECTION_CLIPBOARD)
emit_main_context(channel, SPICE_MAIN_CLIPBOARD,
- cb->type, cb->data, msg->size - sizeof(VDAgentClipboard));
+ cb->type, data, size);
+ if (data != cb->data)
+ g_free(data);
break;
}
case VD_AGENT_CLIPBOARD_GRAB:
@@ -2554,13 +2605,27 @@ void spice_main_clipboard_notify(SpiceMainChannel *channel,
* Since: 0.6
**/
void spice_main_clipboard_selection_notify(SpiceMainChannel *channel, guint selection,
- guint32 type, const guchar *data, size_t size)
+ guint32 type, const guchar *_data, size_t _size)
{
+ const guchar *data = _data;
+ gsize size = _size;
+
g_return_if_fail(channel != NULL);
g_return_if_fail(SPICE_IS_MAIN_CHANNEL(channel));
+ SpiceMainChannelPrivate *c = channel->priv;
+
+ if (type == VD_AGENT_CLIPBOARD_UTF8_TEXT) {
+ if (CLIENT_LINEEND_CRLF && GUEST_LINEEND_LF(c))
+ data = convert_lineend(data, &size, "\r\n", "\n");
+ if (CLIENT_LINEEND_LF && GUEST_LINEEND_CRLF(c))
+ data = convert_lineend(data, &size, "\n", "\r\n");
+ }
agent_clipboard_notify(channel, selection, type, data, size);
spice_channel_wakeup(SPICE_CHANNEL(channel), FALSE);
+
+ if (data != _data)
+ g_free((guchar *)data);
}
/**

53
0004-usb-widget-fix-gtk2-Python-bindings.patch
View File

@ -1,53 +0,0 @@
From 3010789e722f4a0fce62d7f172ad8134e1c5e866 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= <marcandre.lureau@redhat.com>
Date: Tue, 6 Aug 2013 12:09:18 +0200
Subject: [spice-gtk] usb-widget: fix gtk2 Python bindings
The Python bindings generator failed to bind the USB widget, because of
the object/class declaration. The declaration was circumventing the
deprecated errors when compiling with GTK_DISABLE_DEPRECATED. We used
to need that because of broken gtk+ headers, but it is no longer
necessary since 15bd7ceba1434b5d710bfd16078044f30693467b.
(cherry picked from commit a7565265532f6abec0fd3c6d843683b58eee070a)
---
gtk/usb-device-widget.h | 12 ++----------
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/gtk/usb-device-widget.h b/gtk/usb-device-widget.h
index 3920990..b68cc6b 100644
--- a/gtk/usb-device-widget.h
+++ b/gtk/usb-device-widget.h
@@ -37,14 +37,6 @@ typedef struct _SpiceUsbDeviceWidget SpiceUsbDeviceWidget;
typedef struct _SpiceUsbDeviceWidgetClass SpiceUsbDeviceWidgetClass;
typedef struct _SpiceUsbDeviceWidgetPrivate SpiceUsbDeviceWidgetPrivate;
-#if GTK_CHECK_VERSION(3,0,0)
-typedef struct _GtkBox _SpiceGtkBox;
-typedef struct _GtkBoxClass _SpiceGtkBoxClass;
-#else
-typedef struct _GtkVBox _SpiceGtkBox;
-typedef struct _GtkVBoxClass _SpiceGtkBoxClass;
-#endif
-
/**
* SpiceUsbDeviceWidget:
*
@@ -52,7 +44,7 @@ typedef struct _GtkVBoxClass _SpiceGtkBoxClass;
*/
struct _SpiceUsbDeviceWidget
{
- _SpiceGtkBox parent;
+ GtkVBox parent;
/*< private >*/
SpiceUsbDeviceWidgetPrivate *priv;
@@ -67,7 +59,7 @@ struct _SpiceUsbDeviceWidget
*/
struct _SpiceUsbDeviceWidgetClass
{
- _SpiceGtkBoxClass parent_class;
+ GtkVBoxClass parent_class;
/* signals */
void (*connect_failed) (SpiceUsbDeviceWidget *widget,

27
0005-glib-compat-g_slist_free_full-pass-the-right-ptr-to-.patch
View File

@ -1,27 +0,0 @@
From f5dfea28a3b909e51bcb544c399b02bd3ff65df7 Mon Sep 17 00:00:00 2001
From: Hans de Goede <hdegoede@redhat.com>
Date: Mon, 19 Aug 2013 16:51:58 +0200
Subject: [PATCH spice-gtk] glib-compat: g_slist_free_full: pass the right ptr
to destroy (rhbz#997893)
The destroy function passed to g_slist_free_full should be passed the elements
data pointer, not the element itself.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
gtk/glib-compat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gtk/glib-compat.c b/gtk/glib-compat.c
index 21be1f6..c3bb8e6 100644
--- a/gtk/glib-compat.c
+++ b/gtk/glib-compat.c
@@ -88,7 +88,7 @@ g_slist_free_full(GSList *list,
if (free_func) {
for (el = list; el ; el = g_slist_next(el)) {
- free_func(el);
+ free_func(el->data);
}
}

53
0006-gtk-channel-cursor.c-add-cursor_type_to_string-for-d.patch
View File

@ -1,53 +0,0 @@
From 630d4d0b2cddefa85aeab796b1859a9d65aaec5d Mon Sep 17 00:00:00 2001
From: Alon Levy <alevy@redhat.com>
Date: Wed, 28 Aug 2013 16:01:15 +0300
Subject: [PATCH 1/2] gtk/channel-cursor.c: add cursor_type_to_string for
debugging
---
gtk/channel-cursor.c | 22 ++++++++++++++++++++--
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/gtk/channel-cursor.c b/gtk/channel-cursor.c
index 99e7a48..41ad696 100644
--- a/gtk/channel-cursor.c
+++ b/gtk/channel-cursor.c
@@ -325,6 +325,23 @@ static void display_cursor_unref(display_cursor *cursor)
g_free(cursor);
}
+static const char *cursor_type_to_string(int type)
+{
+ switch (type) {
+ case SPICE_CURSOR_TYPE_MONO:
+ return "mono";
+ case SPICE_CURSOR_TYPE_ALPHA:
+ return "alpha";
+ case SPICE_CURSOR_TYPE_COLOR32:
+ return "color32";
+ case SPICE_CURSOR_TYPE_COLOR16:
+ return "color16";
+ case SPICE_CURSOR_TYPE_COLOR4:
+ return "color4";
+ }
+ return "unknown";
+}
+
static display_cursor *set_cursor(SpiceChannel *channel, SpiceCursor *scursor)
{
SpiceCursorChannelPrivate *c = SPICE_CURSOR_CHANNEL(channel)->priv;
@@ -343,8 +360,9 @@ static display_cursor *set_cursor(SpiceChannel *channel, SpiceCursor *scursor)
if (scursor->flags & SPICE_CURSOR_FLAGS_NONE)
return NULL;
- CHANNEL_DEBUG(channel, "%s: type %d, %" PRIx64 ", %dx%d", __FUNCTION__,
- hdr->type, hdr->unique, hdr->width, hdr->height);
+ CHANNEL_DEBUG(channel, "%s: type %s(%d), %" PRIx64 ", %dx%d", __FUNCTION__,
+ cursor_type_to_string(hdr->type), hdr->type, hdr->unique,
+ hdr->width, hdr->height);
if (scursor->flags & SPICE_CURSOR_FLAGS_FROM_CACHE) {
item = cache_find(&c->cursors, hdr->unique);
--
1.8.3.1

50
0007-gtk-channel-cursor-copy-spicec-hack-RHBZ-998529.patch
View File

@ -1,50 +0,0 @@
From c4428fd886ca344fb77a684028e181236873b05e Mon Sep 17 00:00:00 2001
From: Alon Levy <alevy@redhat.com>
Date: Wed, 28 Aug 2013 16:14:16 +0300
Subject: [PATCH 2/2] gtk/channel-cursor: copy spicec hack, RHBZ #998529
flip -> unsupported by x11, since XCreatePixmapCursor has no invert
functionality, only a mask, shape, background and foreground colors. Use
this checkerboard hack to get some contrast for cursors in the guest
that relied on invert for the same contrast.
---
gtk/channel-cursor.c | 23 ++++++++++++++++++-----
1 file changed, 18 insertions(+), 5 deletions(-)
diff --git a/gtk/channel-cursor.c b/gtk/channel-cursor.c
index 41ad696..e4a996b 100644
--- a/gtk/channel-cursor.c
+++ b/gtk/channel-cursor.c
@@ -259,11 +259,24 @@ static void mono_cursor(display_cursor *cursor, const guint8 *data)
for (x = 0; x < cursor->hdr.width; x++, dest += 4) {
if (and[x/8] & bit) {
if (xor[x/8] & bit) {
- /* flip -> hmm? */
- dest[0] = 0x00;
- dest[1] = 0x00;
- dest[2] = 0x00;
- dest[3] = 0x80;
+ /*
+ * flip -> unsupported by x11, since XCreatePixmapCursor has
+ * no invert functionality, only a mask, shape, background and
+ * foreground colors. Use this checkerboard hack to get some
+ * contrast for cursors in the guest that relied on invert for
+ * the same contrast.
+ */
+ if ((x ^ y) & 1) {
+ dest[0] = 0x30;
+ dest[1] = 0x30;
+ dest[2] = 0x30;
+ dest[3] = 0xc0;
+ } else {
+ dest[0] = 0x50;
+ dest[1] = 0x50;
+ dest[2] = 0x50;
+ dest[3] = 0x30;
+ }
} else {
/* unchanged -> transparent */
dest[0] = 0x00;
--
1.8.3.1

2
sources
View File

@ -1 +1 @@
9ef6b7f2f368b7966029b7834b7c2bec spice-gtk-0.20.tar.bz2
290d1ad5ab8f1e5708fa2549dde2e024 spice-gtk-0.21.tar.bz2

28
spice-gtk.spec
View File

@ -12,8 +12,8 @@
#define _version_suffix
Name: spice-gtk
Version: 0.20
Release: 6%{?dist}
Version: 0.21
Release: 1%{?dist}
Summary: A GTK+ widget for SPICE clients
Group: System Environment/Libraries
@ -21,13 +21,6 @@ License: LGPLv2+
URL: http://spice-space.org/page/Spice-Gtk
#VCS: git:git://anongit.freedesktop.org/spice/spice-gtk
Source0: http://www.spice-space.org/download/gtk/%{name}-%{version}%{?_version_suffix}.tar.bz2
Patch1: 0001-Add-spice_channel_string_to_type-to-map-files.patch
Patch2: 0002-smartcard-Handle-VCARD_EMUL_INIT_ALREADY_INITED.patch
Patch3: 0003-channel-main-Convert-text-line-endings-if-necessary-.patch
Patch4: 0004-usb-widget-fix-gtk2-Python-bindings.patch
Patch5: 0005-glib-compat-g_slist_free_full-pass-the-right-ptr-to-.patch
Patch6: 0006-gtk-channel-cursor.c-add-cursor_type_to_string-for-d.patch
Patch7: 0007-gtk-channel-cursor-copy-spicec-hack-RHBZ-998529.patch
BuildRequires: intltool
BuildRequires: gtk2-devel >= 2.14
@ -153,16 +146,10 @@ if [ -n '%{?_version_suffix}' ]; then
mv spice-gtk-%{version}%{?_version_suffix} spice-gtk-%{version}
fi
pushd spice-gtk-%{version}
%patch0001 -p1
%patch0002 -p1
%patch0003 -p1
%patch0004 -p1
%patch0005 -p1
%patch0006 -p1
%patch0007 -p1
find . -name '*.stamp' | xargs touch
popd
#pushd spice-gtk-%{version}
#%patch1 -p1
#find . -name '*.stamp' | xargs touch
#popd
%if %{with_gtk3}
cp -a spice-gtk-%{version} spice-gtk3-%{version}
@ -282,6 +269,9 @@ rm -rf %{buildroot}%{_datadir}/pkgconfig/spice-protocol.pc
%{_bindir}/spicy-stats
%changelog
* Wed Sep 18 2013 Marc-André Lureau <marcandre.lureau@redhat.com> - 0.21-1
- Update to spice-gtk 0.21
* Fri Sep 13 2013 Christophe Fergeau <cfergeau@redhat.com> 0.20-6
- Add misc upstream patches fixing various 0.20 bugs