Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Elnode work on Emacs 26.1 - MELPA Version #108

Closed
wants to merge 1 commit into from

Conversation

jcaw
Copy link

@jcaw jcaw commented Jun 1, 2019

This PR is essentially the same as #107, but the changes are targetted at the MELPA branch so the fix will filter out to MELPA.

This fix should be safe. It's the same method used by emacs-web-server to fix the same bug. Digging around, Emacs 25 ignores :nowait when server it t. Emacs 26.1 just formalises this, by throwing an error.

Fixes #106
Closes #105

Initially I thought Emacs 25.1+ was broken, but it looks like this isn't the case - it seems to be exclusive to Emacs 26.1+.

Elnode doesn't work on Emacs 26.1, because it disallows both the `:nowait` and
`:server` parameters. Add a cautious fix for this, which flips `:nowait` to nil
when Emacs is on 26.1 upwards.

This is the same method used by `emacs-http-server` to fix the same bug.

Initial fix:

  eschulte/emacs-web-server#9

26.1-only patch:

  eschulte/emacs-web-server@cafa5b7
@jcaw
Copy link
Author

jcaw commented Jun 8, 2019

Closing due to MELPA updating their recipe: melpa/melpa#6184

Active fork here: https://github.com/jcaw/elnode

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant