[cairo-bugs] [Bug 78709] any2ppm fails to build on architectures without MMU

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu May 15 01:32:53 PDT 2014


https://bugs.freedesktop.org/show_bug.cgi?id=78709

--- Comment #2 from Ravi Nanjundappa <nravi.n at samsung.com> ---
Comment on attachment 99045
  --> https://bugs.freedesktop.org/attachment.cgi?id=99045
Add check for fork() when compiling any2ppm as daemon

I feel conditional checking of HAVE_FORK macro, should be seperated out.
Because in the above patch, HAVE_FORK has been added in the conditional
checking of header file related macros.

How about the below change ? 


diff --git a/test/any2ppm.c b/test/any2ppm.c
index b125496..3e0806f 100644
--- a/test/any2ppm.c
+++ b/test/any2ppm.c
@@ -92,8 +92,10 @@
 #define SOCKET_PATH "./.any2ppm"
 #define TIMEOUT 60000 /* 60 seconds */

+#if HAVE_FORK
 #define CAN_RUN_AS_DAEMON 1
 #endif
+#endif

 #define ARRAY_LENGTH(A) (sizeof (A) / sizeof (A[0]))

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cairographics.org/archives/cairo-bugs/attachments/20140515/114b8aef/attachment.html>


More information about the cairo-bugs mailing list