[glib] g_test_trap_fork: don't blow away the SIGCHLD handler



commit 38859e8e2930f12bc5885c1ce9191aa3f73b8110
Author: Dan Winship <danw gnome org>
Date:   Wed Dec 19 15:59:55 2012 -0500

    g_test_trap_fork: don't blow away the SIGCHLD handler
    
    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 |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)
---
diff --git a/glib/gtestutils.c b/glib/gtestutils.c
index 4089758..897e927 100644
--- a/glib/gtestutils.c
+++ b/glib/gtestutils.c
@@ -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));


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]