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

ob-async does not specify correct minimum required Emacs version - seq-random-elt missing #55

Open
3 tasks done
ZelphirKaltstahl opened this issue Apr 3, 2019 · 2 comments

Comments

@ZelphirKaltstahl
Copy link

ZelphirKaltstahl commented Apr 3, 2019

Checklist

  • I followed the troubleshooting guide and it did not address my issue
  • I've included a link to a Github Gist with the full contents of my troubleshooting.org buffer
  • (hopefully) Each section below is filled in and includes sufficient detail to reproduce the issue

Expected Behavior

  • The code should run, give a sensible result, and run asynchronously.
  • Versions should be specified correctly for the ob-async package.

Actual Behavior

  • The code is not run without error and the result is weird.
  • Versions are specified as follows in the melpa-stable package summary inside Emacs: Requires: async-1.9, org-9.0.1, emacs-24.4, dash-2.14.1, however, emacs-24.4 does not seem to have seq-random-elt.

Steps to Reproduce

  1. Install Emacs in version: GNU Emacs 25.2.2 (x86_64-pc-linux-gnu, GTK+ Version 3.22.21) of 2017-09-22, modified by Debian
  2. Install ob-async via list-packages and from melpa-stable
  3. Create an Org-mode file.
  4. Add the following source block:
#+BEGIN_SRC elisp :async :results output
(async-start
   ;; What to do in the child process
   (lambda ()
     (message "This is a test")
     (sleep-for 3)
     222)

   ;; What to do when it finishes
   (lambda (result)
     (message "Async process done, result should be 222: %s" result)))
#+END_SRC
  1. Try to run the code.
  2. See the error message in the *messages* buffer.

Troubleshooting Artifacts

I created an emacs.stackexchange question about this problem: https://emacs.stackexchange.com/questions/48658/ob-async-issue-seq-random-elt-not-defined

Here is my troubleshooting.org file: https://gist.githubusercontent.com/ZelphirKaltstahl/68ab4f8cbd558fb6eea71e211adb1269/raw/cdd614e8069f7bd3a6da64c1d594e07414b49f6f/troubleshooting.org

@astahlman
Copy link
Owner

Will try and find some time to look at this in-depth in a few days, but I don't immediately understand the issue.

ob-async does not directly call seq-random-elt, and I don't see any reference to it in https://github.com/jwiegley/emacs-async either.

The integration tests run against Emacs 24.5, too, so if the package were totally broken on Emacs 24 the tests should surface it (in theory).

@ZelphirKaltstahl
Copy link
Author

Could it be a dependency of a dependency issue?

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

No branches or pull requests

2 participants