apps/gdalserver.c uses signal definitions without signal.h

[gdt]

According to POSIX, one must include signal.h when using signal(3).

[Kurt Schwehr]

+1 on the bug, but I am working on an alternate patch

[Kurt Schwehr]

Propose change to cpl_spawn.cpp rather than port.

cpl_spawn.cpp

@@ -34 +34 @@
 #include "cpl_string.h"
 #include "cpl_multiproc.h"
 
+#ifdef SIGPIPE
+#  include <csignal>
+#endif
+
 #define PIPE_BUFFER_SIZE    4096
 
 #define IN_FOR_PARENT   0

[Kurt Schwehr]

Sorry, I see you meant gdalserver.c. Ignore my last. I've applied the patch to gdalserver.c _{but doing it by hand.}

r31286

The line numbers in the patch are bad. If not all systems define SIGPIPE, we may need a #ifdefined around

     if( !bFork )
         signal(SIGPIPE, SIG_IGN);

[Even Rouault]

branches/2.0 r31326 "Add include of signal.h for SIGPIPE and signal(). (patch by Greg Troxel, #6193)"