Re: bug #92117

[Colin Walters <walters@cis.ohio-state.edu>]

[ I'm on the list, so no need for a CC ]

"Davide G. M. Salvetti" <salve@debian.org> writes:

> myself (auctex, mailcrypt) and possibly someone other use:
> 
> /usr/share/$FLAVOR/site-lisp/$PACKAGE/CompilationLog.gz
> 
> since ages.  That kind of stuff may prove useful on bug reports
> times, when you could ask the user to send it to you: better keep it
> somewhere; /usr/share/$FLAVOR/site-lisp/$PACKAGE/ seems quite like
> the best place, since it get generated on installation time (i.e.,
> you don't need /var/ at all).

This does indeed seem to be ideal.  I'd like to see this in the Emacs
policy.  Here's a patch, which also includes a typo fix.

cd ~/tmp/
diff -u -L /usr/doc/emacsen-common/debian-emacs-policy.gz -L /home/walters/tmp/debian-emacs-policy /tmp/jka-comxMCSos /home/walters/tmp/debian-emacs-policy
--- /usr/doc/emacsen-common/debian-emacs-policy.gz
+++ /home/walters/tmp/debian-emacs-policy
@@ -200,6 +200,14 @@
        /etc/<emacs>/site-start.d
        /usr/share/<emacs>/site-lisp/<package-name>
 
+     The site-start.d directory is intended for setting up autoloads
+     and the like for the package.
+       
+     Typically, byte-compiled versions of the code are placed in the
+     latter directory.  If packages do byte-compilation, they should
+     place the compilation log in
+     /usr/share/<emacs>/site-lisp/<package-name>/CompilationLog.gz.
+
   D) Each add-on package must declare relevant dependencies on other
      packages (including other add-on packages).  Note that add-on
      packages should not depend on emacsen-common directly, but rather
@@ -270,7 +278,7 @@
 
 Examples (I've been told that there may be some bugs in these
 examples, but I haven't had time to check through them yet.  So don't
-take them as gospel at the moment.  I suggest that you loook at
+take them as gospel at the moment.  I suggest that you look at
 existing packages for examples of working code.  I will probably be
 revamping these soon, but I don't want to do it now since there are
 probably about to be some changes that will invalidate some of this):
@@ -345,7 +353,8 @@
        cp ${el_path_list} ${elc_dir}
 
        # Byte compile them
-       ${FLAVOR} ${byte_compile_options} ${el_path_list}
+       ${FLAVOR} ${byte_compile_options} ${el_path_list} 2>&1 | gzip -9qf >/usr/share/$FLAVOR/site-lisp/$PACKAGE/CompilationLog.gz
+       echo "
 
        # remove the redundant .el files
        # presumes that any .el files in the <flavor> dir are trash.

Diff finished at Wed Jun 27 20:33:39