Re: bootstrap not regenerating some *loaddefs

[Juanma Barranquero]

On Wed, Jun 18, 2008 at 17:20, Stefan Monnier <address@hidden> wrote:

> The patch seem to be doing the right thing

This one is better; it also removes some autoloads and defvars that
are currently added to loaddefs.el on Windows and which would be
unnecessary now.

   Juanma



2008-06-18  Juanma Barranquero  <address@hidden>

        * makefile.w32-in (bootstrap-clean-CMD, bootstrap-clean-SH):
        Don't copy ldefs-boot.el over loaddefs.el.
        (bootstrap-clean): Remove loaddefs.el and don't depend on it.
        (loaddefs.el-SH, loaddefs.el-CMD): Don't add autoloads and defvars
        to loaddefs.el; they are not needed now.  Add coding cookie.


Index: lisp/makefile.w32-in
===================================================================
RCS file: /sources/emacs/emacs/lisp/makefile.w32-in,v
retrieving revision 1.81
diff -u -2 -r1.81 makefile.w32-in
--- lisp/makefile.w32-in        6 May 2008 08:05:36 -0000       1.81
+++ lisp/makefile.w32-in        18 Jun 2008 11:57:13 -0000
@@ -130,35 +130,24 @@
        echo ";;; loaddefs.el --- automatically extracted autoloads" > $@
        echo ";;" >> $@; echo ";;; Code:" >> $@
-       echo "(autoload 'define-minor-mode \"easy-mmode\")" >>$@
-       echo "(autoload 'define-ccl-program \"ccl\")" >>$@
-       echo "(autoload 'regexp-opt \"regexp-opt\")" >>$@
-       echo "(autoload 'string-to-list \"mule-util\")" >>$@
-       echo "(autoload 'define-derived-mode \"derived\")" >>$@
-       echo "(autoload 'encoded-kbd-mode \"encoded-kb\")" >>$@
-       echo "(defvar cvs-global-menu nil)" >>$@
        echo "" >> $@
-       echo ";;; Local Variables:" >> $@
-       echo ";;; version-control: never" >> $@
-       echo ";;; no-byte-compile: t" >> $@
-       echo ";;; no-update-autoloads: t" >> $@
-       echo ";;; End:" >> $@
+       echo ";; Local Variables:" >> $@
+       echo ";; version-control: never" >> $@
+       echo ";; no-byte-compile: t" >> $@
+       echo ";; no-update-autoloads: t" >> $@
+       echo ";; coding: utf-8" >> $@
+       echo ";; End:" >> $@
        echo ";;; loaddefs.el ends here" >> $@

 loaddefs.el-CMD:
        echo ;;; loaddefs.el --- automatically extracted autoloads> $@
+       echo ;;>> $@
        echo ;;; Code:>> $@
-       echo (autoload 'define-minor-mode "easy-mmode")>> $@
-       echo (autoload 'define-ccl-program "ccl")>> $@
-       echo (autoload 'regexp-opt "regexp-opt")>> $@
-       echo (autoload 'string-to-list "mule-util")>> $@
-       echo (autoload 'define-derived-mode "derived")>> $@
-       echo (autoload 'encoded-kbd-mode "encoded-kb")>> $@
-       echo (defvar cvs-global-menu nil)>> $@
        echo.>> $@
-       echo ;;; Local Variables:>> $@
-       echo ;;; version-control: never>> $@
-       echo ;;; no-byte-compile: t>> $@
-       echo ;;; no-update-autoloads: t>> $@
-       echo ;;; End:>> $@
+       echo ;; Local Variables:>> $@
+       echo ;; version-control: never>> $@
+       echo ;; no-byte-compile: t>> $@
+       echo ;; no-update-autoloads: t>> $@
+       echo ;; coding: utf-8>> $@
+       echo ;; End:>> $@
        echo ;;; loaddefs.el ends here>> $@

@@ -385,14 +374,10 @@

 # Need separate version for sh and native cmd.exe
-# Note that bootstrap-clean-$(SHELLTYPE) copies ldefs-boot.el to loaddefs.el,
-# and thus the almost-empty loaddefs.el crafted by the $(lisp)/loaddefs.el
-# target can NOT be built _after_ bootstrap-clean-$(SHELLTYPE) does its
-# thing, or else an empty loaddefs.el will overwrite the full one.
-bootstrap-clean: $(lisp)/loaddefs.el
+bootstrap-clean:
+       - $(DEL) $(lisp)/loaddefs.el
        $(MAKE) $(MFLAGS) bootstrap-clean-$(SHELLTYPE)

 bootstrap-clean-CMD:
 #      if exist "$(EMACS)" $(MAKE) $(MFLAGS) autoloads
-       cp $(lisp)/ldefs-boot.el $(lisp)/loaddefs.el
        -for %%f in (. $(WINS)) do for %%g in (%%f\*.elc) do @$(DEL) %%g

@@ -400,5 +385,4 @@
 #      if test -f "$(EMACS)"; then $(MAKE) $(MFLAGS) autoloads; fi
 #      -rm -f $(lisp)/*.elc $(lisp)/*/*.elc
-       cp $(lisp)/ldefs-boot.el $(lisp)/loaddefs.el
        -for dir in . $(WINS); do rm -f $$dir/*.elc; done