> 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. > Is a ## some code requires at least Vista enough? (the double "#" make automake strip the comment before producing Makefile.in). > > > $(COMMON_CFLAGS) \ > > $(GLIB2_CFLAGS) \ > > $(GOBJECT2_CFLAGS) \ Frediano _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel