astahlman / ob-async

Asynchronous src_block execution for org-babel
343 stars 32 forks source link

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

Open ZelphirKaltstahl opened 5 years ago

ZelphirKaltstahl commented 5 years ago

Checklist

Expected Behavior

Actual Behavior

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
  5. Try to run the code.
  6. 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 commented 5 years ago

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 commented 5 years ago

Could it be a dependency of a dependency issue?