Hi,<br><br>Thanks for looking though the patch :)<br><br>On Wed, Sep 29, 2010 at 5:46 PM, Alexander Larsson <span dir="ltr">&lt;<a href="mailto:alexl@redhat.com">alexl@redhat.com</a>&gt;</span> wrote:<br><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">On Sun, 2010-09-26 at 14:57 +0200, Attila Sukosd wrote:<br>
&gt; Hi All,<br>
&gt;<br>
&gt; I finally had a bit of time to gather the changes to the spice client<br>
&gt; in order to get it working under Mac.<br>
<br>
</div>Good stuff. I landed some of the cleanup from this patch as:<br>
<a href="http://lists.freedesktop.org/archives/spice-devel/2010-September/001249.html" target="_blank">http://lists.freedesktop.org/archives/spice-devel/2010-September/001249.html</a><br>
<br>
Also, I did a new version of the select patch that further simplifies<br>
things:<br>
<a href="http://lists.freedesktop.org/archives/spice-devel/2010-September/001264.html" target="_blank">http://lists.freedesktop.org/archives/spice-devel/2010-September/001264.html</a><br>
Waiting for review on it...<br>
<br>
Further review:<br>
<br>
Why the GPL change? All other headers are LGPL 2.1 anyway, and we want<br>
the client LGPL anyway so it can be librariefied.<br></blockquote><div><br>Yea, sorry bout the licensing change, I guess the old sources had some different licensing.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div class="im"><br>
+#if defined(__APPLE__) || defined(__MACH__)<br>
+#ifndef MSG_NOSIGNAL<br>
+#define MSG_NOSIGNAL SO_NOSIGPIPE<br>
+#endif<br>
+#endif<br>
<br>
</div>This is not right. SO_NOSIGPIPE is not used in the same way as<br>
MSG_NOSIGNAL.<br>
See e.g.<br>
<a href="http://lists.policyd.org/pipermail/devel/2007-September/000468.html" target="_blank">http://lists.policyd.org/pipermail/devel/2007-September/000468.html</a><br>
<br></blockquote><div>Hmm, thats good to know.<br><br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
In general we want to avoid sprinkling __LINUX__ or __APPLE__ defines in<br>
the &quot;unixy&quot; code (its hard to avoid for win32 though). Instead we want<br>
to check things in configure and use proper HAVE_FOO checks, etc.<br>
<br>
So, for instance we&#39;d have HAVE_ALSA (or maybe USE_ALSA in this case<br>
since we might have multiple detected sound system in any particular<br>
system), rather than __LINUX__ checks. And similar for the thread stuff.<br>
<br></blockquote><div>Yes, I agree, but it was just much easier to play around with it, without having to modify too much in the autoconf stuff.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

Anyway, if you rebase your patch based on the stuff thats landed things<br>
should look much easier for you to handle.<br>
<br>
</blockquote></div><br><br>