From: Matthias Clasen Date: Fri, 1 Apr 2005 21:40:43 +0000 (+0000) Subject: Add a note regarding waitpid(-1). X-Git-Url: http://git.openbox.org/?a=commitdiff_plain;h=0914ea84e3c7a48fa59f686c4182787df3fe5320;p=dana%2Fcg-glib.git Add a note regarding waitpid(-1). 2005-04-01 Matthias Clasen * glib/gmain.c (g_child_watch_source_new): Add a note regarding waitpid(-1). (g_child_watch_source_init_multi_threaded): (g_child_watch_source_init_single): Don't use SA_RESTART, since it causes problems on at least one platform. (#168352) --- diff --git a/ChangeLog b/ChangeLog index 575dad53..6c5cb25f 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,6 +1,14 @@ +2005-04-01 Matthias Clasen + + * glib/gmain.c (g_child_watch_source_new): Add a note regarding + waitpid(-1). + (g_child_watch_source_init_multi_threaded): + (g_child_watch_source_init_single): Don't use SA_RESTART, + since it causes problems on at least one platform. (#168352) + 2005-03-30 Steve Murphy - * configure.in: Added "rw" to ALL_LINGUAS. + * configure.in: Added "rw" to ALL_LINGUAS. 2005-03-30 Sven Neumann diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10 index 575dad53..6c5cb25f 100644 --- a/ChangeLog.pre-2-10 +++ b/ChangeLog.pre-2-10 @@ -1,6 +1,14 @@ +2005-04-01 Matthias Clasen + + * glib/gmain.c (g_child_watch_source_new): Add a note regarding + waitpid(-1). + (g_child_watch_source_init_multi_threaded): + (g_child_watch_source_init_single): Don't use SA_RESTART, + since it causes problems on at least one platform. (#168352) + 2005-03-30 Steve Murphy - * configure.in: Added "rw" to ALL_LINGUAS. + * configure.in: Added "rw" to ALL_LINGUAS. 2005-03-30 Sven Neumann diff --git a/ChangeLog.pre-2-12 b/ChangeLog.pre-2-12 index 575dad53..6c5cb25f 100644 --- a/ChangeLog.pre-2-12 +++ b/ChangeLog.pre-2-12 @@ -1,6 +1,14 @@ +2005-04-01 Matthias Clasen + + * glib/gmain.c (g_child_watch_source_new): Add a note regarding + waitpid(-1). + (g_child_watch_source_init_multi_threaded): + (g_child_watch_source_init_single): Don't use SA_RESTART, + since it causes problems on at least one platform. (#168352) + 2005-03-30 Steve Murphy - * configure.in: Added "rw" to ALL_LINGUAS. + * configure.in: Added "rw" to ALL_LINGUAS. 2005-03-30 Sven Neumann diff --git a/ChangeLog.pre-2-8 b/ChangeLog.pre-2-8 index 575dad53..6c5cb25f 100644 --- a/ChangeLog.pre-2-8 +++ b/ChangeLog.pre-2-8 @@ -1,6 +1,14 @@ +2005-04-01 Matthias Clasen + + * glib/gmain.c (g_child_watch_source_new): Add a note regarding + waitpid(-1). + (g_child_watch_source_init_multi_threaded): + (g_child_watch_source_init_single): Don't use SA_RESTART, + since it causes problems on at least one platform. (#168352) + 2005-03-30 Steve Murphy - * configure.in: Added "rw" to ALL_LINGUAS. + * configure.in: Added "rw" to ALL_LINGUAS. 2005-03-30 Sven Neumann diff --git a/glib/gmain.c b/glib/gmain.c index 4cc320b6..4d262e26 100644 --- a/glib/gmain.c +++ b/glib/gmain.c @@ -3565,7 +3565,7 @@ g_child_watch_source_init_single (void) action.sa_handler = g_child_watch_signal_handler; sigemptyset (&action.sa_mask); - action.sa_flags = SA_RESTART | SA_NOCLDSTOP; + action.sa_flags = SA_NOCLDSTOP; sigaction (SIGCHLD, &action, NULL); } @@ -3674,6 +3674,11 @@ g_child_watch_source_init (void) * (see g_spawn_close_pid()) @pid must not be closed while the * source is still active. Typically, you will want to call * g_spawn_close_pid() in the callback function for the source. + * + * Note further that using g_child_watch_source_new() is not + * compatible with calling waitpid(-1) in + * the application. Calling waitpid() for individual pids will + * still work fine. * * Return value: the newly-created child watch source *