g_test_trap_fork: don't blow away the SIGCHLD handler
authorDan Winship <danw@gnome.org>
Wed, 19 Dec 2012 20:59:55 +0000 (15:59 -0500)
committerDan Winship <danw@gnome.org>
Mon, 13 May 2013 16:10:52 +0000 (12:10 -0400)
Not sure why it was doing this, but it's not necessary (all of glib's
tests pass fine without it), and it breaks tests that try to use
g_spawn_sync() or GChildWatchSource after doing a g_test_trap_fork().

https://bugzilla.gnome.org/show_bug.cgi?id=679683

glib/gtestutils.c

index 4089758..897e927 100644 (file)
@@ -2203,7 +2203,6 @@ g_test_trap_fork (guint64        usec_timeout,
   test_trap_clear();
   if (pipe (stdout_pipe) < 0 || pipe (stderr_pipe) < 0 || pipe (stdtst_pipe) < 0)
     g_error ("failed to create pipes to fork test program: %s", g_strerror (errno));
-  signal (SIGCHLD, SIG_DFL);
   test_trap_last_pid = fork ();
   if (test_trap_last_pid < 0)
     g_error ("failed to fork test program: %s", g_strerror (errno));