> > I think the right fix is for someone with a clean checkout of the lisp > > files to generate and check in a new version of ldefs-boot.el > > Why do you think that will solve the problem?
It won't. Your analysis is more complete than mine. It just seemed to me that flyspell-mode-map shouldn't be needed to build Emacs and putting back autoload cookies that Stefan had just removed wasn't the answer. Nick > If I understand things correctly, bootstrap fails because of this > piece of code in flyspell.el: > > ;;;###autoload > (add-minor-mode 'flyspell-mode > 'flyspell-mode-line-string > flyspell-mode-map > nil > 'flyspell-mode) > > It results in this piece of code in the generated loaddefs.el: > > (add-minor-mode (quote flyspell-mode) (quote flyspell-mode-line-string) > flyspell-mode-map nil (quote flyspell-mode)) > > Since flyspell-mode-map isn't autoloaded (and thus not defined in > loaddefs.el), this gives an error when loaddefs.el is loaded by > loadup.el. So even a clean checkout will fail to bootstrap. > > Removing the add-minor-mode call from flyspell.el and using > define-minor-mode to implement flyspell-mode does solve the problem > for me. _______________________________________________ Emacs-devel mailing list Emacs-devel@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-devel
