From 38859e8e2930f12bc5885c1ce9191aa3f73b8110 Mon Sep 17 00:00:00 2001 From: Dan Winship Date: Wed, 19 Dec 2012 15:59:55 -0500 Subject: [PATCH] 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 file changed, 1 deletion(-) diff --git a/glib/gtestutils.c b/glib/gtestutils.c index 4089758c6..897e927e6 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));