Re: WCONTINUED is not portable

2014-03-24 Thread Jakub Jelinek
On Sun, Mar 23, 2014 at 03:18:42PM -0700, Ian Lance Taylor wrote:
 On Sun, Mar 23, 2014 at 3:50 AM, Martin Husemann mar...@duskware.de wrote:
  The flag WCONTINUED for waitpid() is not portable.
 
  The attached patch just defines it to 0 if missing.
 
 I don't understand why that code passes either WUNTRACED or
 WCONTINUED.  Why not just pass 0?

Note the patch Martin has posted is not needed, I've committed a change
to define WCONTINUED to 0 if not defined several days ago as obvious.

Adding Honza in the loop why he chose to use WUNTRACED | WCONTINUED.

Jakub


WCONTINUED is not portable

2014-03-23 Thread Martin Husemann
The flag WCONTINUED for waitpid() is not portable.

The attached patch just defines it to 0 if missing.

Martin
--- gcc/lto/lto.c.orig  2014-02-24 23:58:44.0 +0100
+++ gcc/lto/lto.c   2014-03-21 16:43:05.0 +0100
@@ -2470,6 +2470,11 @@ do_stream_out (char *temp_filename, lto_
 
 /* Wait for forked process and signal errors.  */
 #ifdef HAVE_WORKING_FORK
+
+#ifndef WCONTINUED
+#defineWCONTINUED  0
+#endif
+
 static void
 wait_for_child ()
 {


Re: WCONTINUED is not portable

2014-03-23 Thread Ian Lance Taylor
On Sun, Mar 23, 2014 at 3:50 AM, Martin Husemann mar...@duskware.de wrote:
 The flag WCONTINUED for waitpid() is not portable.

 The attached patch just defines it to 0 if missing.

I don't understand why that code passes either WUNTRACED or
WCONTINUED.  Why not just pass 0?

Ian