Nicolas Goaziou (2016-05-07 15:50 +0300) wrote:

> Hello,
>
> Alex Kost <alez...@gmail.com> writes:
>
>> Thanks!  Sorry I didn't build asymptote (as I don't have a wish to build
>> gigabytes of texlive), but I looked at the source and I see there are
>> "autoloads" cookies in "<asymptote>/base/asy-mode.el", so it would also
>> be good to generate "asymptote-autoloads.el" file.
>
> Indeed.
>
>> For example, you may look at (gnu packages emacs) module to see how
>> "autoloads" file are generated using 'emacs-generate-autoloads'
>> procedure.  I write some notes below, that seems appropriate to me, but
>> I'm not going to test the patch because of texlive (sorry again).  I
>> hope other people will do.
>
> Fair enough.
>
> Here is an updated patch with suggested changes. Thank you for the review.

Thanks!  Looks good to me, so if there will be no other comments in
several days, I will commit it.  Did you check that the resulting
"/share/emacs/site-lisp" directory contains elisp files (including
"asymptote-autoloads.el")?

-- 
Alex

Reply via email to