Re: make bootstrap fails due to grep autoload

[Kim F. Storm]

Stefan Monnier <address@hidden> writes:

>> ;;;###autoload
>> (define-compilation-mode grep-mode "Grep"
>
> Why would anyone want to autoload `grep-mode'?

I don't know.

It was added by yourself when you installed this change:

2004-03-11  Daniel Pfeiffer  <address@hidden>

        * progmodes/grep.el (kill-grep): Move here from compile.el
        (grep-error, grep-hit-face, grep-error-face)
        (grep-mode-font-lock-keywords): New variables.
        (grep-regexp-alist): Simplify regexp and add `binary' case.
        (grep-mode): New mode.
        (grep-process-setup): Simplify.


> Maybe we want to put
>
>         (autoload 'grep-mode "grep" nil t)
>
> in compile.el so that code that does (require 'compile) in order to get
> `grep-mode' still works, but the autoload has nothing to do in loaddefs.el

Seems unnecessary.  

grep-mode is new in 21.4, and in 21.4 grep-mode is only used in grep.el.

-- 
Kim F. Storm <address@hidden> http://www.cua.dk