[virt-tools-list] [PATCH virt-viewer] session-spice: Remove spice-gtk version checks
Fabiano Fidêncio
fabiano at fidencio.org
Thu Apr 2 07:18:51 UTC 2015
On Thu, Apr 2, 2015 at 8:09 AM, Pavel Grunt <pgrunt at redhat.com> wrote:
> Since 77ac0d8892837a117f9ca10020c1ac7f1944fca7 virt-viewer requires
> spice-gtk v0.28
> ---
> src/virt-viewer-session-spice.c | 11 -----------
> 1 file changed, 11 deletions(-)
>
> diff --git a/src/virt-viewer-session-spice.c b/src/virt-viewer-session-spice.c
> index a0acba6..921e613 100644
> --- a/src/virt-viewer-session-spice.c
> +++ b/src/virt-viewer-session-spice.c
> @@ -44,10 +44,6 @@
> #include "gbinding.c"
> #endif
>
> -#ifndef SPICE_GTK_CHECK_VERSION
> -#define SPICE_GTK_CHECK_VERSION(x, y, z) 0
> -#endif
> -
> G_DEFINE_TYPE (VirtViewerSessionSpice, virt_viewer_session_spice, VIRT_VIEWER_TYPE_SESSION)
>
>
> @@ -546,14 +542,12 @@ virt_viewer_session_spice_main_channel_event(SpiceChannel *channel G_GNUC_UNUSED
> const GError *error = NULL;
> g_debug("main channel: auth failure (wrong username/password?)");
>
> -#if SPICE_GTK_CHECK_VERSION(0, 26, 0)
> {
> error = spice_channel_get_error(channel);
> username_required = g_error_matches(error,
> SPICE_CLIENT_ERROR,
> SPICE_CLIENT_ERROR_AUTH_NEEDS_PASSWORD_AND_USERNAME);
> }
> -#endif
>
> if (self->priv->pass_try > 0)
> g_signal_emit_by_name(session, "session-auth-failed",
> @@ -594,7 +588,6 @@ virt_viewer_session_spice_main_channel_event(SpiceChannel *channel G_GNUC_UNUSED
> break;
> }
> case SPICE_CHANNEL_ERROR_CONNECT:
> -#if SPICE_GTK_CHECK_VERSION(0, 23, 21)
> {
> const GError *error = spice_channel_get_error(channel);
>
> @@ -619,10 +612,6 @@ virt_viewer_session_spice_main_channel_event(SpiceChannel *channel G_GNUC_UNUSED
> g_signal_emit_by_name(session, "session-disconnected", error ? error->message : NULL);
> }
> }
> -#else
> - g_debug("main channel: failed to connect");
> - g_signal_emit_by_name(session, "session-disconnected", NULL);
> -#endif
> break;
> case SPICE_CHANNEL_ERROR_IO:
> case SPICE_CHANNEL_ERROR_LINK:
> --
> 2.3.4
>
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list
ACK!
--
Fabiano Fidêncio
More information about the virt-tools-list
mailing list