Re: ldefs-boot.el

[Luc Teirlinck]

Thien-Thi Nguyen wrote:

   > # bootstrap should always work with ldefs-boot.el.
   > 
   > No, it does not always work.  (How could it?)  It only works if you
   > update ldefs-boot whenever needed.

   the "should" in the last sentence of the makefile frag implies the last
   sentence you wrote ("it only works...").  perhaps adding that sentence to
   the makefile would be sufficient to nudge the implication to explanation?

I suggest the following exoansion of the comment in lisp/Makefile.in.
I can install if it looks OK.

===File ~/lisp-Makefile.in-diff=============================
*** Makefile.in 07 Aug 2005 10:23:06 -0500      1.65
--- Makefile.in 13 Aug 2005 18:29:24 -0500      
***************
*** 218,227 ****
  # Build loaddefs.el to make sure it's up-to-date.  If it's not, that
  # might lead to errors during the bootstrap because something fails to
  # autoload as expected.  If there is no emacs binary, then we can't
! # build autoloads yet.  In that case we have to use ldefs-boot.el;
! # bootstrap should always work with ldefs-boot.el.  (Because
! # loaddefs.el is an automatically generated file, we don't want to
! # store it in the source repository).
  
  bootstrap-prepare:
        if test -x $(EMACS); then                               \
--- 218,230 ----
  # Build loaddefs.el to make sure it's up-to-date.  If it's not, that
  # might lead to errors during the bootstrap because something fails to
  # autoload as expected.  If there is no emacs binary, then we can't
! # build autoloads yet.  In that case we have to use ldefs-boot.el.
! # Bootstrap should always work with ldefs-boot.el.  Therefore,
! # whenever a new autoload cookie gets added that is necessary during
! # bootstrapping, ldefs-boot.el should be updated by overwriting it with
! # an up-to-date copy of loaddefs.el that is uncorrupted by
! # local changes.  (Because loaddefs.el is an automatically generated
! # file, we don't want to store it in the source repository).
  
  bootstrap-prepare:
        if test -x $(EMACS); then                               \
============================================================