[virt-tools-list] [PATCH 1/2] Fix spice includes

Pavel Grunt pgrunt at redhat.com
Tue Mar 22 10:57:46 UTC 2016


Hi,

you need to bump spice-gtk dependency to v0.31, because these headers
are not in v0.30 and virt-viewer currently depends on v0.30.

Ack with the change

Thanks,
Pavel

On Tue, 2016-03-15 at 11:48 -0300, Eduardo Lima (Etrunko) wrote:
> Recent spice commit requires that only spice-client.h or
> spice-client-gtk.h should be included directly. As a result,
> compilation is now throwing warnings like:
> 
> warning: #warning "Only <spice-client.h> can be included directly" [-
> Wcpp]
> warning: #warning "Only <spice-client-gtk.h> can be included
> directly" [-Wcpp]
> 
> Signed-off-by: Eduardo Lima (Etrunko) <etrunko at redhat.com>
> ---
>  src/virt-viewer-display-spice.c | 2 +-
>  src/virt-viewer-display-spice.h | 3 +--
>  src/virt-viewer-session-spice.c | 5 +----
>  src/virt-viewer-session-spice.h | 3 +--
>  4 files changed, 4 insertions(+), 9 deletions(-)
> 
> diff --git a/src/virt-viewer-display-spice.c b/src/virt-viewer-
> display-spice.c
> index 5114833..4abab2c 100644
> --- a/src/virt-viewer-display-spice.c
> +++ b/src/virt-viewer-display-spice.c
> @@ -25,7 +25,7 @@
>  #include <config.h>
>  
>  #include <math.h>
> -#include <spice-audio.h>
> +#include <spice-client-gtk.h>
>  
>  #include <glib/gi18n.h>
>  
> diff --git a/src/virt-viewer-display-spice.h b/src/virt-viewer-
> display-spice.h
> index 3c30270..598a1b7 100644
> --- a/src/virt-viewer-display-spice.h
> +++ b/src/virt-viewer-display-spice.h
> @@ -25,8 +25,7 @@
>  #define _VIRT_VIEWER_DISPLAY_SPICE_H
>  
>  #include <glib-object.h>
> -#include <spice-widget.h>
> -#include <spice-audio.h>
> +#include <spice-client.h>
>  
>  #include "virt-viewer-display.h"
>  #include "virt-viewer-session-spice.h"
> diff --git a/src/virt-viewer-session-spice.c b/src/virt-viewer-
> session-spice.c
> index dc0c1ff..f7f2dc9 100644
> --- a/src/virt-viewer-session-spice.c
> +++ b/src/virt-viewer-session-spice.c
> @@ -24,12 +24,9 @@
>  
>  #include <config.h>
>  
> -#include <spice-audio.h>
>  #include <glib/gi18n.h>
>  
> -#include <spice-option.h>
> -#include <spice-util.h>
> -#include <spice-client.h>
> +#include <spice-client-gtk.h>
>  
>  #include <usb-device-widget.h>
>  #include "virt-viewer-file.h"
> diff --git a/src/virt-viewer-session-spice.h b/src/virt-viewer-
> session-spice.h
> index 95bdcdf..aa1f7e8 100644
> --- a/src/virt-viewer-session-spice.h
> +++ b/src/virt-viewer-session-spice.h
> @@ -25,8 +25,7 @@
>  #define _VIRT_VIEWER_SESSION_SPICE_H
>  
>  #include <glib-object.h>
> -#include <spice-widget.h>
> -#include <spice-audio.h>
> +#include <spice-client.h>
>  
>  #include "virt-viewer-session.h"
>  




More information about the virt-tools-list mailing list