Re: [PATCH spice-server 3/33 v2] build: Detect Windows build and change some definitions

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

 



On Wed, Jan 02, 2019 at 02:58:45PM +0000, Frediano Ziglio wrote:
> Windows needs some specific setting to use network.
> 
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  configure.ac | 20 +++++++++++++++++++-
>  1 file changed, 19 insertions(+), 1 deletion(-)
> 
> Changes since v1:
> - all changes in configure.ac
> 
> diff --git a/configure.ac b/configure.ac
> index 96e4e66c..afe0017d 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -68,6 +68,20 @@ case $host_cpu in
>          SPICE_WARNING([spice-server on non-x86_64 architectures has not been extensively tested])
>  esac
>  
> +AC_MSG_CHECKING([for native Win32])
> +case "$host_os" in
> +     *mingw*|*cygwin*)
> +        os_win32=yes
> +        dnl Some code requires at least Vista
> +        AC_DEFINE([_WIN32_WINNT], [0x600], [Minimal Win32 version])]

Can this be part of the commit which adds code which needs Vista
behaviour? This way git blame would tell which code started to require at least
Vista.

> +        ;;
> +     *)
> +        os_win32=no
> +        ;;
> +esac
> +AC_MSG_RESULT([$os_win32])
> +AM_CONDITIONAL([OS_WIN32],[test "$os_win32" = "yes"])
> +
>  dnl =========================================================================
>  dnl Check optional features
>  SPICE_CHECK_SMARTCARD
> @@ -153,6 +167,9 @@ AC_CHECK_LIB(rt, clock_gettime, LIBRT="-lrt")
>  AC_SUBST(LIBRT)
>  
>  AS_VAR_APPEND([SPICE_NONPKGCONFIG_LIBS], [" -pthread $LIBM $LIBRT"])
> +AS_IF([test "x$os_win32" = "xyes"], [
> +    AS_VAR_APPEND([SPICE_NONPKGCONFIG_LIBS], [" -lws2_32"])
> +])

I'd be tempted to also add this when it becomes really needed?

Christophe

>  
>  SPICE_REQUIRES=""
>  
> @@ -175,7 +192,8 @@ PKG_CHECK_MODULES([GOBJECT2], [gobject-2.0 >= $GLIB2_REQUIRED])
>  AS_VAR_APPEND([SPICE_REQUIRES], [" gobject-2.0 >= $GLIB2_REQUIRED"])
>  
>  #used only by tests
> -PKG_CHECK_MODULES([GIO_UNIX], [gio-unix-2.0 >= $GLIB2_REQUIRED])
> +AS_IF([test "x$os_win32" != "xyes"],
> +      PKG_CHECK_MODULES([GIO_UNIX], [gio-unix-2.0 >= $GLIB2_REQUIRED]))
>  
>  PIXMAN_REQUIRED=0.17.7
>  PKG_CHECK_MODULES(PIXMAN, pixman-1 >= $PIXMAN_REQUIRED)
> -- 
> 2.20.1
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

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