[PATCH 21.5] Do not insert timestamps in .elc files

"Stephen J. Turnbull" <[email protected]> Fri, 26 Sep 2014 09:57:18 +0900
Newsgroups gmane.emacs.xemacs.patches
Message-ID <[email protected]>
Jerry James writes:
 > PATCH 21.5
 > 
 > Over on xemacs-beta, Stephen J. Turnbull <[email protected]> wrote:
 > > Why not just make it a commit-time constant?  That mailbox is great!
 > 
 > :-) I hoped somebody would like that.  My favorite of all time,
 > though, was the email address of Patrice Godefroid when he was at Bell
 > Labs: [email protected].

$ host olympus.net
CNAME att.com

<wink/>

On the DMARC front, I've been hanging out with a guy who styles
himself [email protected], which is arguably bigger than god@att
these days. :-)  (Except of course Googlers get @google addresses.)

 > Sure.  The use case of interest is compiling on systems that vary only
 > in CPU architecture.  They will have the same sets of software
 > installed, with exactly the same versions, etc.  This change will make
 > it possible to verify that we're getting identical results in such
 > cases, which means the files can be shared across all architectures
 > (we can produce a "noarch" package, to use RPM terminology).

OK that seems like a good use case.

 >    (with-string-as-buffer-contents ""
 > -    (insert "\n;;; compiled by "
 > -    (or (and (boundp 'user-mail-address) user-mail-address)
 > - (concat (user-login-name) "@" (system-name)))

I'm not sure I agree with this, though.  Why not have the build do

    xemacs -eval '(setq user-mail-address "[email protected]")'

I think it's useful to be able to easily distinguish between packaged
compiles and local ones: "That doesn't sound like our build.  Check
the first line of the .elc; ours always say 'compiled by
[email protected]'."  Time is different, there's no point that I
can see in setting time to a constant.

 > -    " on "
 > -    (current-time-string) "\n;;; from file " filename "\n")
 > +    (insert "\n;;; compiled from file " filename "\n")
 >      (insert ";;; emacs version " emacs-version ".\n")
 >      (insert ";;; bytecomp version " byte-compile-version "\n;;; "
 >      (cond
 > 
 > -- 
 > Jerry James
 > http://www.jamezone.org/
 > 
 > _______________________________________________
 > XEmacs-Patches mailing list
 > [email protected]
 > http://lists.xemacs.org/mailman/listinfo/xemacs-patches