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

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

 



Hi

On Fri, Dec 21, 2018 at 4:03 PM Frediano Ziglio <fziglio@xxxxxxxxxx> wrote:
>
> Windows needs some specific setting to use network.
>
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

with neat below
Reviewed-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>

> ---
>  configure.ac       | 18 +++++++++++++++++-
>  server/Makefile.am |  1 +
>  2 files changed, 18 insertions(+), 1 deletion(-)
>
> diff --git a/configure.ac b/configure.ac
> index 96e4e66c..a2a81cef 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -68,6 +68,18 @@ 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
> +        ;;
> +     *)
> +        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 +165,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"])
> +])
>
>  SPICE_REQUIRES=""
>
> @@ -175,7 +190,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)
> diff --git a/server/Makefile.am b/server/Makefile.am
> index 34ec22ad..5009d197 100644
> --- a/server/Makefile.am
> +++ b/server/Makefile.am
> @@ -6,6 +6,7 @@ check-valgrind:
>
>  AM_CPPFLAGS =                                  \
>         -DSPICE_SERVER_INTERNAL                 \
> +       -D_WIN32_WINNT=0x600                    \

I would add a comment in commit message or here that this require Vista API.

I have a preference to put those defines in a common top headers and
use #ifndef (so they can be easily overriden with CFLAGS). I am not
sure if we have a one though.


>         $(COMMON_CFLAGS)                        \
>         $(GLIB2_CFLAGS)                         \
>         $(GOBJECT2_CFLAGS)                      \
> --
> 2.17.2
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel



--
Marc-André Lureau
_______________________________________________
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]