From: Eli Zaretskii <eliz@gnu.org>
To: Leo Liu <sdl.web@gmail.com>
Cc: emacs-devel@gnu.org
Subject: Re: master f995fbd: * lisp/server.el (server-name): Add autoload cookie. (Bug#23576)
Date: Fri, 20 May 2016 10:46:05 +0300 [thread overview]
Message-ID: <83mvnl894y.fsf@gnu.org> (raw)
In-Reply-To: <m237pdw6dq.fsf@gmail.com> (message from Leo Liu on Fri, 20 May 2016 15:11:45 +0800)
> From: Leo Liu <sdl.web@gmail.com>
> Date: Fri, 20 May 2016 15:11:45 +0800
>
> On 2016-05-20 08:51 +0300, Eli Zaretskii wrote:
> > Sorry, I don't think I understand what that means. What duplication
> > do you allude to here?
>
> For example if you autoload server-name, the whole
>
> (defcustom server-name "server"
> "The name of the Emacs server, if this Emacs process creates one.
> The command `server-start' makes use of this. It should not be
> changed while a server is running."
> :group 'server
> :type 'string
> :version "23.1")
>
> is duplicated to the loaddefs file and now you have two definitions in
> two separate files.
And why is that a problem? loaddefs.el is auto-generated, so no one
needs to maintain it.
> > I added the autoload for a reason for which I didn't see a better
> > solution. If you can suggest a better solution that allows users to
> > specify the name before starting the first server, please do.
>
> Get rid of the set-variable.
Sorry, no. I use that command myself a lot, so I see no reason why it
should be avoided.
next prev parent reply other threads:[~2016-05-20 7:46 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20160518194708.27363.47377@vcs.savannah.gnu.org>
[not found] ` <20160518194708.86FE0220157@vcs.savannah.gnu.org>
2016-05-19 16:44 ` master f995fbd: * lisp/server.el (server-name): Add autoload cookie. (Bug#23576) Glenn Morris
2016-05-19 17:13 ` Eli Zaretskii
2016-05-20 4:07 ` Leo Liu
2016-05-20 5:51 ` Eli Zaretskii
2016-05-20 7:11 ` Leo Liu
2016-05-20 7:46 ` Eli Zaretskii [this message]
2016-05-20 8:48 ` Dmitry Gutov
2016-05-20 9:41 ` Eli Zaretskii
2016-05-20 12:12 ` Stefan Monnier
2016-05-20 13:34 ` Eli Zaretskii
2016-05-20 18:29 ` Stefan Monnier
2016-05-20 18:59 ` Eli Zaretskii
2016-05-26 3:01 ` Stefan Monnier
2016-05-26 3:36 ` Stefan Monnier
2016-05-26 15:38 ` Eli Zaretskii
2016-05-27 19:51 ` Stefan Monnier
2016-05-28 7:11 ` Eli Zaretskii
2016-06-17 16:42 ` Stefan Monnier
2016-06-17 20:35 ` Clément Pit--Claudel
2016-06-17 22:35 ` Stefan Monnier
2016-06-18 2:20 ` Clément Pit--Claudel
2016-06-18 2:38 ` Dmitry Gutov
2016-06-18 18:09 ` Stefan Monnier
2016-06-18 20:05 ` Dmitry Gutov
2016-06-23 13:45 ` Clément Pit--Claudel
2016-06-23 13:48 ` Clément Pit--Claudel
2016-06-23 14:27 ` Dmitry Gutov
2016-06-23 14:48 ` Clément Pit--Claudel
2016-06-23 14:30 ` Dmitry Gutov
2016-06-23 15:35 ` Stefan Monnier
2016-06-23 15:52 ` Dmitry Gutov
2016-06-24 14:01 ` Stefan Monnier
2016-07-01 13:33 ` Robert Weiner
2016-07-01 15:51 ` Stefan Monnier
2016-05-20 17:30 ` Glenn Morris
2016-05-20 17:47 ` Eli Zaretskii
2016-05-20 18:15 ` Dmitry Gutov
2016-05-20 18:55 ` Eli Zaretskii
2016-05-21 4:25 ` Leo Liu
2016-05-21 6:52 ` Eli Zaretskii
2016-05-21 7:05 ` Dmitry Gutov
2016-05-21 7:32 ` Eli Zaretskii
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=83mvnl894y.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=sdl.web@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.