Chris Marusich (2016-04-04 03:58 +0300) wrote: > Hi, > > The attached patches add some useful modules for reading RFCs (and STDs) > in Emacs.
Overall the patches look good to me, thanks! But there is a problem with both packages: the URL of the origin does not point to a versioned file, so when the author will touch the source, the hash will be changed and our package will not work anymore. I suppose there are no mirrors where we can fetch the "constant" source code, so we probably have to live with this, but it is a potential problem-maker. > To take full advantage of them (e.g., read RFCs from a local > directory), you need to make some straightforward modifications to your > ~/.emacs file which are described in comments within the elisp modules. I think that for ffap-rfc-space, no changes in .emacs need to be done, as the generated "ffap-rfc-space-autoloads.el" file contains the required modification for ffap command. Unluckily, rfcview.el package does not have an autoload cookie for adjusting 'auto-mode-alist' as it is described there, so this modification has to be done by a user. But this is easy to fix, so if you wish to contact with the maintainer you may ask him to make this change (as it is done, for example, in csv-mode¹). ¹ http://git.savannah.gnu.org/cgit/emacs/elpa.git/tree/packages/csv-mode/csv-mode.el#n354 -- Alex
