Re: problem of define-minor-mode while bootstrapping

[Kenichi Handa <handa@m17n.org>]

In article <200209191346.g8JDkdq07175@rum.cs.yale.edu>, "Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu> writes:

>>  Here's the last few lines of define-minor-mode.
>>  
>>         ;; If the mode is global, call the function according to the default.
>>         ,(if globalp
>>  	    `(if (and load-file-name ,mode)
>>  		 (eval-after-load load-file-name '(,mode 1)))))))
>>  
>>  Could someone please fix this problem at first?

> Update your sandbox, I've fixed it a few days ago AFAIK,

Do you mean this change?

	* emacs-lisp/easy-mmode.el (define-minor-mode): Add a :require arg.
	Don't call the function during init if mode is on by default.

***************
*** 224,230 ****
  			       (symbol-value ',keymap-sym))))
         
         ;; If the mode is global, call the function according to the default.
!        ,(if globalp
  	    `(if (and load-file-name ,mode)
  		 (eval-after-load load-file-name '(,mode 1)))))))
  \f
--- 227,233 ----
  			       (symbol-value ',keymap-sym))))
         
         ;; If the mode is global, call the function according to the default.
!        ,(if (and globalp (null init-value))
  	    `(if (and load-file-name ,mode)
  		 (eval-after-load load-file-name '(,mode 1)))))))


Could you please install it in RC too if that causes no
problem?

But, it seems that, with this, if :init-value is t, the
function to do various settings to make the mode work is
never called.

---
Ken'ichi HANDA
handa@etl.go.jp