[glib] g_test_trap_fork: don't blow away the SIGCHLD handler
- From: Dan Winship <danw src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [glib] g_test_trap_fork: don't blow away the SIGCHLD handler
- Date: Mon, 13 May 2013 16:11:22 +0000 (UTC)
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]