bug#8137: Requiring a file during bytecompilation runs that file's kill-emacs-hook.

[Chong Yidong]

Juanma Barranquero <lekktu@gmail.com> writes:

> compiling code requiring a file that sets kill-emacs-hook runs that
> hook, for example
>
> ;;; bug.el
> (require 'saveplace)
> ;;; end of bug.el
>
> emacs -batch -Q -f batch-byte-compile bug.el
> Wrote c:/emacs/repo/bug.elc
> Wrote c:/my_home/.emacs-places

This doesn't seem like it's related to Glenn's change.  From a brief
look at saveplace.el, it looks like the problem isn't limited to byte
compilation: the saveplace file would be written anytime Emacs is run
noninteractively while it's loaded (e.g. from a user's init file).
Probably saveplace.el needs to be changed so that it doesn't do anything
when noninteractive.

There are nine other places in Emacs that add to kill-emacs-hook at
top-level, which ought to be checked too.