On 29 June 2010 14:54:36 UTC+1, Peter Pentchev <[email protected]> wrote:
> I've added the attached patch to my Subversion repo; I admit that I'm not
> very well versed in e-lisp, so any suggestions / reproaches would be
> welcome :)

I think the lines you've added are fine, but need to be added to
/etc/emacs.d/site-lisp/50gforth.el, not gforth.el, because as far as I
can see, the Debian Emacs routines don't automatically use autoload
(though some individual packages do). This solution is used, for
example, by inform-mode.

However, if you want to really be nice, then you should add the lines
to gforth.el, preceded by

;;;###autoload

so that they work with autoload, and then also add the lines to
50gforth.el, so they are actually loaded at startup by Debian Emacs.
This has two advantages:

1. You can supply the patch to gforth.el to upstream, and it will
improve gforth.el for all Emacs users.

2. If Debian ever starts using autoload in all Emacs extension
packages, then the necessary change to gforth.el has already been
made.

> BTW, is there some kind of a list of the filename extensions "registered"
> by the various Emacs modes in Debian?  Meaning, is there a way for me to
> check if I'm stepping on anybody's toes by grabbing .fs, .4th, and .fth?

Not as far as I know; that's a question for the Emacs maintainers. You
may in any case want to check whether what I said above is correct.

-- 
http://rrt.sc3d.org



--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to