nicferrier / elnode

evented io webserver right inside your emacs.
http://nicferrier.github.com/elnode
GNU General Public License v3.0
477 stars 49 forks source link

[bug] use `aget` without first `(require 'assoc)` #79

Closed eschulte closed 11 years ago

eschulte commented 11 years ago

elnode uses aget without first (require 'assoc) leading to an undefined function error.

This was found by Torsten on the Org-mode mailing list.

eschulte commented 11 years ago

Actually it seems assoc is deprecated, so it'd probably be better to just add a definition for aget directly.

nicferrier commented 11 years ago

Ermmm.....

https://github.com/nicferrier/elnode/blob/master/elnode.el#L59

Yes, assoc is deprecated. Doesn't mean it can't be used though. I have options in kv that I could probably do but I don't have unlimited time.

eschulte commented 11 years ago

Thanks for adding that require, it is not present in the ELPA version of elnode that installs by default (at least as of a week or two ago). These bugs are the result of people trying to install and use org-ehtml through ELPA, a process which I'd like to make painless for non-elisp types.

nicferrier commented 11 years ago

Odd. Clearly I haven't done a release for a while as that has been fixed for months.

notifications@github.com writes:

Thanks for adding that require, it is not present in the ELPA version of elnode that installs by default (at least as of a week or two ago). These bugs are the result of people trying to install and use org-ehtml through ELPA, a process which I'd like to make painless for non-elisp types.


Reply to this email directly or view it on GitHub: https://github.com/nicferrier/elnode/issues/79#issuecomment-26731346