> > On Sat, Dec 22, 2018 at 09:56:38AM -0500, Frediano Ziglio wrote: > > > > > 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 Like AC_DEFINE([_WIN32_WINNT], [0x600], [Minimal Win32 version])] ?? I suppose as long as is defined is fine. The changes are pretty minimal, just some minor additional declarations. > > > > + ;; > > > > + *) > > > > + 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). > > Why not AC_DEFINE _WIN32_WINNT when appropriate in configure.ac? > > Christophe > It was just easier to do it. > > > > > > > > > $(COMMON_CFLAGS) \ > > > > $(GLIB2_CFLAGS) \ > > > > $(GOBJECT2_CFLAGS) \ > > Frediano _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel