[macruby-changes] [4452] MacRuby/trunk/io.c

[source_changes at macosforge.org]

Revision: 4452
          http://trac.macosforge.org/projects/ruby/changeset/4452
Author:   lsansonetti at apple.com
Date:     2010-08-23 18:13:08 -0700 (Mon, 23 Aug 2010)
Log Message:
-----------
properly reap child processes before closing their IO objects (patch contributed by jhemmelg at gmail.com)

Modified Paths:
--------------
    MacRuby/trunk/io.c

Modified: MacRuby/trunk/io.c
===================================================================
--- MacRuby/trunk/io.c	2010-08-23 20:20:07 UTC (rev 4451)
+++ MacRuby/trunk/io.c	2010-08-24 01:13:08 UTC (rev 4452)
@@ -354,11 +354,8 @@
 	io_struct->write_fd = -1;
     }
     if (io_struct->pid != -1) {
-	// Don't commit seppuku!
 	rb_last_status_set(0, io_struct->pid);
-	if (io_struct->pid != 0 && io_struct->pid != getpid()) {
-	    kill(io_struct->pid, SIGTERM);
-	}
+	rb_syswait(io_struct->pid);
 	io_struct->pid = -1;
     }
     if (io_struct->fd != -1 && io_struct->read_fd == -1
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macruby-changes/attachments/20100823/9ab0aec0/attachment.html>