[Spice-devel] [PATCH] usbredirserver: Make compile under MacOS

Christophe Fergeau cfergeau at redhat.com
Tue Jan 29 09:23:24 UTC 2019


On Mon, Jan 28, 2019 at 02:28:36PM -0500, Frediano Ziglio wrote:
> > On Wed, Jan 23, 2019 at 10:09:25AM +0000, Frediano Ziglio wrote:
> > > This fixes Gitlab issue #9
> > > (cfr https://gitlab.freedesktop.org/spice/usbredir/issues/9).
> > > 
> > > Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> > > ---
> > >  usbredirserver/usbredirserver.c | 7 +++++++
> > >  1 file changed, 7 insertions(+)
> > > 
> > > diff --git a/usbredirserver/usbredirserver.c
> > > b/usbredirserver/usbredirserver.c
> > > index 6aa2740..429985a 100644
> > > --- a/usbredirserver/usbredirserver.c
> > > +++ b/usbredirserver/usbredirserver.c
> > > @@ -43,6 +43,13 @@
> > >  
> > >  #define SERVER_VERSION "usbredirserver " PACKAGE_VERSION
> > >  
> > > +#if !defined(SOL_TCP) && defined(IPPROTO_TCP)
> > > +#define SOL_TCP IPPROTO_TCP
> > > +#endif
> > > +#if !defined(TCP_KEEPIDLE) && defined(TCP_KEEPALIVE)
> > > +#define TCP_KEEPIDLE TCP_KEEPALIVE
> > > +#endif
> > 
> > Might be better to restrict this to OSX in case a system decides to
> > use TCP_KEEPALIVE, but with a different meaning?
> > 
> 
> So
> 
> #if !defined(TCP_KEEPIDLE) && defined(TCP_KEEPALIVE) && defined(__APPLE__)

Yep, or even just #if defined(__APPLE__)

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20190129/acb5dc06/attachment.sig>


More information about the Spice-devel mailing list