Re: [PATCH spice-streaming-agent 1/4] configure.ac: remove header check for stream-device.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> The pkg-config version check should be enough to ensure we have the
> correct version of spice-protocol which contains this header.
> 
> Signed-off-by: Lukáš Hrázký <lhrazky@xxxxxxxxxx>

Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

I think this was here just for 0.12.13, we forget to remove the
additional check, now 0.12.14 has the header.

I would a bit rephrase:

"The pkg-config version check is enough to ensure we have the
correct version of spice-protocol which contains this header."

Frediano

> ---
>  configure.ac | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 1d86200..8795dae 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -31,10 +31,6 @@ dnl Check deps
>  SPICE_PROTOCOL_MIN_VER=0.12.14
>  PKG_CHECK_MODULES([SPICE_PROTOCOL], [spice-protocol >=
>  $SPICE_PROTOCOL_MIN_VER])
>  AC_SUBST([SPICE_PROTOCOL_MIN_VER])
> -SAVE_CFLAGS="$CFLAGS"
> -CFLAGS="$SPICE_PROTOCOL_CFLAGS $CFLAGS"
> -AC_CHECK_HEADER([spice/stream-device.h],,[AC_MSG_ERROR([Could not locate
> spice-protocol stream-device.h header])])
> -CFLAGS="$SAVE_CFLAGS"
>  
>  PKG_CHECK_MODULES(X11, x11)
>  PKG_CHECK_MODULES(XFIXES, xfixes)
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]