GNU bug report logs - #24218
25.1.50; server-name, server-socket-dir and daemon start through systemd socket

Previous Next

Package: emacs;

Reported by: Vladimir Lomov <lomov.vl <at> gmail.com>

Date: Sat, 13 Aug 2016 16:37:02 UTC

Severity: normal

Found in version 25.1.50

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

Bug is archived. No further changes may be made.

Full log


Message #14 received at 24218 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 24218 <at> debbugs.gnu.org, matthew <at> mattleach.net, lomov.vl <at> gmail.com
Subject: Re: bug#24218: 25.1.50;
 server-name, server-socket-dir and daemon start through, systemd
 socket
Date: Mon, 08 Jan 2018 05:50:25 +0200
> From: Paul Eggert <eggert <at> cs.ucla.edu>
> Date: Sun, 7 Jan 2018 13:55:04 -0800
> Cc: 24218 <at> debbugs.gnu.org, Vladimir Lomov <lomov.vl <at> gmail.com>
> 
> Matthew proposed a patch for this here:
> 
> https://lists.gnu.org/r/emacs-devel/2017-12/msg00903.html
> 
> which I am attaching in git form (see first attached patch). I also 
> propose the second attached patch as a followup. It makes some of the 
> newly-added features "private" (since they don't seem to be generally 
> useful) and moves the low-level initialization code from emacs.c to 
> process.c which is a more-natural home for it.

Thanks.  I'm not an expert on this stuff, so I have only minor
comments:

  . Please update the :version tag of server-name, to reflect the fact
    that its default value changes
  . This needs a NEWS entry, and I think the description of
    server-name in the manual should be updated to mention this
    facility
  . internal--external-sockname should be documented in the ELisp
    manual (a better name that doesn't use both "internal" and
    "external" will be appreciated)




This bug report was last modified 7 years and 159 days ago.

Previous Next


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