unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Paul Eggert <eggert@cs.ucla.edu>
To: Eli Zaretskii <eliz@gnu.org>
Cc: p.stephani2@gmail.com, emacs-devel@gnu.org
Subject: Re: [PATCH PING] Honor 'SOURCE_DATE_EPOCH' when generating autoloads.
Date: Tue, 29 Dec 2015 12:33:26 -0800	[thread overview]
Message-ID: <5682EE16.8040607@cs.ucla.edu> (raw)
In-Reply-To: <834mf1dtk5.fsf@gnu.org>

Eli Zaretskii wrote:
> We could come up with something even less probable.

We could, yes. It should not be a valid domain name, for example. But whatever 
magic cookie we came up with, we would need to document it so that user code can 
avoid displaying it. And to avoid typos it would be better to make it a named 
constant. And we would need to do something similar, with different named 
constants, for similar issues such as emacs-build-time. And we would never 
eliminate the possibility that someone (if only to be contrary, e.g., because 
they are attacking Emacs) would use the magic-cookie as a host name. All in all, 
it would be a relatively complicated solution to what should be a simple problem.

>> >In this sense, the Dec. 25 change that introduced the system build name into the
>> >output of report-emacs-bug was a misstep.
> No, it isn't.  I needed this information many times when examining bug
> reports, and I'm not going to give it up.

My comment’s “in the sense” referred to the sense of its previous paragraph (not 
quoted above) which talked about deterministic builds; it did not refer to 
building Emacs in general. In some cases it can be useful to have the build 
hostname embedded in the executable even at the cost of deterministic builds. 
However, many people (including myself) prefer deterministic builds, and for 
them embedding the build hostname is counterproductive, and that is why it is 
useful to have an option to suppress it.

> Can I have some minimal
> respect from fellow developers, and some minimal credit that I do
> things for a reason?

Nothing in my commentary was intended to exhibit any disrespect. It was intended 
to be purely technical commentary. If it seemed otherwise, I apologize.

>>> > >It sounds strange to send users to documentation for such a simple issue.
>> >
>> >No matter what we do here, we will need documentation that users can refer to on
>> >occasion. Even simple approaches like nil-if-unknown require some documentation.
>> >More-complicated approaches like magic-cookie-if-unknown would require more of it.
> You are ducking the issue.

I do not see what I am ducking here. Surely you are not advocating leaving the 
behavior undocumented. Are you objecting to the convention of using nil to 
represent lack of information? If so, how about if we use some other symbol, 
such as ‘unknown’? Although that would be a bit more of a pain to document and 
use, it would also be OK and for the reasons discussed above would be better 
than a magic-cookie string.



  reply	other threads:[~2015-12-29 20:33 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-16 17:03 [PATCH] Honor 'SOURCE_DATE_EPOCH' when generating autoloads Ludovic Courtès
2015-11-29 10:44 ` [PATCH PING] " Ludovic Courtès
2015-11-29 16:02   ` Eli Zaretskii
2015-11-29 16:57     ` Ludovic Courtès
2015-11-29 18:12       ` Eli Zaretskii
2015-11-29 20:38         ` Ludovic Courtès
2015-11-30 17:00         ` John Wiegley
2015-11-30 19:30           ` Ludovic Courtès
2015-12-07 18:36             ` Philipp Stephani
2015-12-07 18:58               ` Paul Eggert
2015-12-07 19:00                 ` Philipp Stephani
2015-12-07 19:14                   ` Paul Eggert
2015-12-20 12:48                     ` Philipp Stephani
2015-12-20 16:37                       ` Eli Zaretskii
2015-12-20 18:39                         ` Philipp Stephani
2015-12-20 19:03                           ` Eli Zaretskii
2015-12-20 21:27                           ` Paul Eggert
2015-12-23  1:13                             ` John Wiegley
2015-12-23 16:01                               ` Philipp Stephani
2015-12-23 16:39                                 ` Eli Zaretskii
2015-12-23 16:52                                   ` Philipp Stephani
2015-12-23 17:10                                     ` Eli Zaretskii
2015-12-23 17:38                                       ` Philipp Stephani
2015-12-23 21:09                                         ` Paul Eggert
2015-12-24  3:33                                           ` Eli Zaretskii
2015-12-24  6:54                                             ` Paul Eggert
2015-12-24 16:18                                               ` Eli Zaretskii
2015-12-27  9:53                                     ` Philipp Stephani
2015-12-27 16:10                                       ` Eli Zaretskii
2016-02-29 21:52                                         ` Philipp Stephani
2016-03-01 16:36                                           ` Paul Eggert
2016-03-01 21:46                                             ` Philipp Stephani
2016-03-02 18:24                                               ` Paul Eggert
2016-03-02 18:30                                                 ` Philipp Stephani
2016-03-22 13:30                                                   ` Philipp Stephani
2016-03-22 20:37                                                     ` Paul Eggert
2016-03-22 21:46                                                       ` Philipp Stephani
2016-03-22 21:58                                                         ` Paul Eggert
2016-03-23  8:03                                                         ` Andreas Schwab
2016-04-06 21:29                                                           ` Philipp Stephani
2015-12-27 23:26                                       ` Paul Eggert
2015-12-28 16:26                                         ` Eli Zaretskii
2015-12-28 18:00                                           ` Paul Eggert
2015-12-28 18:23                                             ` Eli Zaretskii
2015-12-29  3:01                                               ` Paul Eggert
2015-12-29 15:38                                                 ` Eli Zaretskii
2015-12-29 16:47                                                   ` Paul Eggert
2015-12-29 17:59                                                     ` Eli Zaretskii
2015-12-29 20:33                                                       ` Paul Eggert [this message]
2015-12-07 23:11               ` Ludovic Courtès
2015-11-30  9:22       ` Alex Kost
2015-11-29 20:22   ` Paul Eggert
2015-11-29 20:42     ` Ludovic Courtès

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5682EE16.8040607@cs.ucla.edu \
    --to=eggert@cs.ucla.edu \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=p.stephani2@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 public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).