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 Thu, Jan 03, 2019 at 12:15:02PM -0500, Frediano Ziglio wrote:
> > 
> > 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.
> > 
> 
> AI_ADDRCONFIG in reds.c which is already in master.

I'd mention it in the comment I guess "AI_ADDRCONFIG and possibly some
other code require at least Vista"

Christophe

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]