Re: [Qemu-devel] [5194] Fix the build on non-Linux systems

[C . W . Betts]

Actually, it builds in 10.5.  Using the 10.4 SDK causes the problems.

On Sep 11, 2008, at 9:26 AM, C.W. Betts wrote:

Now it complains about missing symbols:

Undefined symbols:

  "_sigwaitinfo", referenced from:

      _sigwait_compat in compatfd.o

ld: symbol(s) not found

collect2: ld returned 1 exit status

make: *** [qemu-img] Error 1

This happened both when trying to compile qemu-img and qemu-nbd.

On Sep 11, 2008, at 8:18 AM, Anthony Liguori wrote:

Revision: 5194
         http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=5194
Author:   aliguori
Date:     2008-09-11 14:18:56 +0000 (Thu, 11 Sep 2008)

Log Message:
-----------
Fix the build on non-Linux systems

It turns out, we're never reading from the signalfd() which is causing it to
remain readable forever.  I'll fix this up but I thought I'd commit this fix
in the interim.

Signed-off-by: Anthony Liguori <address@hidden>

Modified Paths:
--------------
   trunk/compatfd.h

Modified: trunk/compatfd.h
===================================================================
--- trunk/compatfd.h 2008-09-10 20:09:22 UTC (rev 5193)
+++ trunk/compatfd.h 2008-09-11 14:18:56 UTC (rev 5194)
@@ -16,14 +16,10 @@

#include <signal.h>

-#if defined(__linux__) && !defined(SYS_signalfd)
-struct signalfd_siginfo {
+struct qemu_signalfd_siginfo {
    uint32_t ssi_signo;
    uint8_t pad[124];
};
-#else
-#include <linux/signalfd.h>
-#endif

int qemu_signalfd(const sigset_t *mask);