emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: newbie hacker question: how to add eval-to-texinfo?


From: Stefan Monnier
Subject: Re: newbie hacker question: how to add eval-to-texinfo?
Date: 03 May 2004 09:12:59 -0400
User-agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.3.50

> I want to add a function called 'eval-to-texi' to emacs (i already
> signed papers for that).

The name does not make it very clear to me what it does.  Could you explain?

> - I suppose texinfo.el is the right way to add it. ┬┐It is right?.

Depends on the patch.  But it's a very minor issue.  Most patches are not
applied as is but are first sent back for a few minor changes: just send
the patch and we'll tell you what needs to be changed, if anything.

> - All functions in texinfo.el start with a 'texinfo-' prefix. May i to
>   set that prefix to 'eval-to-texi' naming it 'texinfo-eval-to-texi'?

Again, depends on the patch.

> - May i to include an autoload for 'eval-to-texi'?

I don't see why not.

> - To make the patch in unidiff format, would be right to generate it
>   using 'cvs diff texinfo.el' on a fresh emacs cvs checkout? Or from
>   the root directory: 'cvs diff lisp/textmodes/texinfo.el'

Same difference.
 
> - May i to add an entry to lisp/ChangeLog and send a patch for it too?

Better to send the ChangeLog entry separately, but in a patch is
also acceptable.  In most cases the patch is wrong (because the date
needs to be changed, the file has changed in the mean time, etc..)
so it has to be inserted by hand anyway.


        Stefan




reply via email to

[Prev in Thread] Current Thread [Next in Thread]