unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [ELPA] Bug in ob-clojure (org 9.0.5)
@ 2017-03-02 20:22 Tim Cross
  0 siblings, 0 replies; only message in thread
From: Tim Cross @ 2017-03-02 20:22 UTC (permalink / raw)
  To: Emacs developers

[-- Attachment #1: Type: text/plain, Size: 1588 bytes --]

Have reported this on the emacs-orgmode list, but as it also affects the
version of org in ELPA (org-20170210), thought I would repeat it here.

The problem is in the org-babel-execute:clojure function. This function has
the following bit of code

(setq result
       (nrepl-dict-get
    (nrepl-sync-request:eval
     expanded (cider-current-connection) (cider-current-session))
    (if (or (member "output" result-params)
        (member "pp" result-params))
        "out"
      "value")))

The problem is in the call to nrepl-sync-request:eval. The documentation
states for this function

(nrepl-sync-request:eval INPUT CONNECTION &optional NS)

Send the INPUT to the nREPL server synchronously. The request is dispatched
via CONNECTION. If NS is non-nil, include it in the request.
Note the last optional argument NS. This is supposed to be a clojure
namespace. However, the org-babel-execute:clojure function is calling this
function with the output from cider-current-session, which returns a unique
ID representing the current session. As a result, the call is returning a
data structure with an error and no output (perhaps some error handling is
required). The returned result is

(dict status (namespace-not-found done error done state state) id 17
session 43e9fd6c-82ed-49fe-9624-0cfc6f56f8b1 changed-namespaces (dict)
repl-type cljclj)

Note the namespace-not-found

Either the argument should be a call to (cider-current-ns) or perhaps it
should just be left out as I don't see how you can pass the namespace as
part of the block evaluation.


-- 
regards,

Tim

--
Tim Cross

[-- Attachment #2: Type: text/html, Size: 2128 bytes --]

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-03-02 20:22 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-02 20:22 [ELPA] Bug in ob-clojure (org 9.0.5) Tim Cross

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).