bug#14474: 24.3.50; Zombie subprocesses (again)

[Michael Heerdegen]

Hi Paul,

> In <http://lists.gnu.org/archive/html/emacs-devel/2013-05/msg00628.html>
> something like the following milder workaround was suggested instead.
> Michael, does this patch work around the bug for your test case?

Have already installed it to trunk?  The issue is fixed for me after
upgrading my emacs-snapshot to

(emacs-version) ==>

GNU Emacs 24.3.50.1 (x86_64-pc-linux-gnu, GTK+ Version 3.8.2)
of 2013-06-03 on dex, modified by Debian


Thanks,

Michael.

>
> === modified file 'src/xterm.c'
> --- src/xterm.c       2013-05-09 14:49:56 +0000
> +++ src/xterm.c       2013-05-28 16:34:44 +0000
> @@ -9897,6 +9897,13 @@ x_term_init (Lisp_Object display_name, c
>  
>          XSetLocaleModifiers ("");
>  
> +     /* If D-Bus is not already configured, inhibit D-Bus autolaunch,
> +        as autolaunch can mess up Emacs's SIGCHLD handler.
> +        FIXME: Rewrite subprocess handlers to use glib's child watchers.
> +        See Bug#14474.  */
> +     if (! egetenv ("DBUS_SESSION_BUS_ADDRESS"))
> +       xputenv ("DBUS_SESSION_BUS_ADDRESS=");
> +
>          /* Emacs can only handle core input events, so make sure
>             Gtk doesn't use Xinput or Xinput2 extensions.  */
>       xputenv ("GDK_CORE_DEVICE_EVENTS=1");