GNU bug report logs - #16599
24.3.50; src/temacs --daemon fails to start

Previous Next

Package: emacs;

Reported by: Alex Bennée <alex.bennee <at> linaro.org>

Date: Thu, 30 Jan 2014 18:36:01 UTC

Severity: minor

Found in version 24.3.50

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex Bennée <kernel-hacker <at> bennee.com>
Cc: schwab <at> suse.de, 16599 <at> debbugs.gnu.org
Subject: bug#16599: src/temacs --daemon fails
Date: Mon, 17 Feb 2014 18:38:29 +0200
> Date: Mon, 17 Feb 2014 18:15:58 +0200
> From: Eli Zaretskii <eliz <at> gnu.org>
> Cc: 16599 <at> debbugs.gnu.org, kernel-hacker <at> bennee.com
> 
> > From: Andreas Schwab <schwab <at> suse.de>
> > Cc: Alex Bennée <kernel-hacker <at> bennee.com>,
> >   16599 <at> debbugs.gnu.org
> > Date: Mon, 17 Feb 2014 17:03:01 +0100
> > 
> > Eli Zaretskii <eliz <at> gnu.org> writes:
> > 
> > >> From: Alex Bennée <kernel-hacker <at> bennee.com>
> > >> Cc: Alex Bennée <kernel-hacker <at> bennee.com>, Eli Zaretskii
> > >>  <eliz <at> gnu.org>, 16599 <at> debbugs.gnu.org, emacs-devel <at> gnu.org
> > >> Date: Mon, 17 Feb 2014 07:40:35 +0000
> > >> 
> > >> Why did the syms_of_emacs() function feel the need to initialise daemon_pipe?
> > >
> > > Because in a dumped Emacs, daemon_pipe might hold a stale value from
> > > before the dumping.
> > 
> > But syms_of_emacs is not called in a dumped Emacs.
> 
> So I guess we should move that line somewhere where it will be called.

Like in the patch below.  Alex, please try this one instead of the one
I sent before.

=== modified file 'src/emacs.c'
--- src/emacs.c	2014-01-20 19:44:56 +0000
+++ src/emacs.c	2014-02-17 16:36:32 +0000
@@ -721,6 +721,9 @@ main (int argc, char **argv)
   /* If we use --chdir, this records the original directory.  */
   char *original_pwd = 0;
 
+  /* Make sure IS_DAEMON starts up as false.  */
+  daemon_pipe[1] = 0;
+
 #if GC_MARK_STACK
   stack_base = &dummy;
 #endif
@@ -2548,7 +2551,4 @@ libraries; only those already known by E
   Vlibrary_cache = Qnil;
   staticpro (&Vlibrary_cache);
 #endif
-
-  /* Make sure IS_DAEMON starts up as false.  */
-  daemon_pipe[1] = 0;
 }





This bug report was last modified 11 years and 101 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.