From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tim Cross Subject: [PATCH] Fix issue with ob-clojure not returning output Date: Fri, 3 Mar 2017 15:25:17 +1100 Message-ID: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary=001a11418316d7022a0549cbee64 Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:55829) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cjemO-0004Vf-13 for emacs-orgmode@gnu.org; Thu, 02 Mar 2017 23:25:21 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cjemM-0005wV-Mj for emacs-orgmode@gnu.org; Thu, 02 Mar 2017 23:25:20 -0500 Received: from mail-qk0-x229.google.com ([2607:f8b0:400d:c09::229]:35790) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1cjemM-0005wD-ID for emacs-orgmode@gnu.org; Thu, 02 Mar 2017 23:25:18 -0500 Received: by mail-qk0-x229.google.com with SMTP id h9so4073544qke.2 for ; Thu, 02 Mar 2017 20:25:18 -0800 (PST) List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: "Emacs-orgmode" To: emacs-orgmode@gnu.org --001a11418316d7022a0549cbee64 Content-Type: multipart/alternative; boundary=001a11418316d702240549cbee62 --001a11418316d702240549cbee62 Content-Type: text/plain; charset=UTF-8 The attached patch is a simple fix for the problem with ob-clojure using an incorrect cider-current-session call instead of cider-current-ns. For background on the issue see https://emacs.stackexchange.com/questions/30857/clojure-code-evaluation-in-org-mode-produces-no-output/31169#31169 Not 100% convinced that the optional namespace argument should even be included - it really doesn't seem to do anything. However, leaving it in seemed like the minimal change needed. Have tested it and at least I now get data back from evaluation of clojure code blocks where previously, you only got the message "block processed with no output". -- regards, Tim -- Tim Cross --001a11418316d702240549cbee62 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
The attached patch is a simple fix for the problem with ob= -clojure using an incorrect cider-current-session call instead of cider-cur= rent-ns. For background on the issue see=C2=A0https://emacs.stackexchange.com/questions/30857/cloj= ure-code-evaluation-in-org-mode-produces-no-output/31169#31169=C2=A0
Not 100% convinced that the optional namespace argument sh= ould even be included - it really doesn't seem to do anything. However,= leaving it in seemed like the minimal change needed.=C2=A0 Have tested it = and at least I now get data back from evaluation of clojure code blocks whe= re previously, you only got the message "block processed with no outpu= t".=C2=A0

--
regards,

Tim
=

--
Tim Cross

--001a11418316d702240549cbee62-- --001a11418316d7022a0549cbee64 Content-Type: text/x-patch; charset=US-ASCII; name="ob-clojure.patch" Content-Disposition: attachment; filename="ob-clojure.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_iztbd8c90 LS0tCiBsaXNwL29iLWNsb2p1cmUuZWwgfCA1ICsrKy0tCiAxIGZpbGUgY2hhbmdlZCwgMyBpbnNl cnRpb25zKCspLCAyIGRlbGV0aW9ucygtKQoKZGlmZiAtLWdpdCBhL2xpc3Avb2ItY2xvanVyZS5l bCBiL2xpc3Avb2ItY2xvanVyZS5lbAppbmRleCBkNDA3MTA1Li5lNTQyYTI5IDEwMDY0NAotLS0g YS9saXNwL29iLWNsb2p1cmUuZWwKKysrIGIvbGlzcC9vYi1jbG9qdXJlLmVsCkBAIC00NCw2ICs0 NCw3IEBACiAKIChkZWNsYXJlLWZ1bmN0aW9uIGNpZGVyLWN1cnJlbnQtY29ubmVjdGlvbiAiZXh0 OmNpZGVyLWNsaWVudCIgKCZvcHRpb25hbCB0eXBlKSkKIChkZWNsYXJlLWZ1bmN0aW9uIGNpZGVy LWN1cnJlbnQtc2Vzc2lvbiAiZXh0OmNpZGVyLWNsaWVudCIgKCkpCisoZGVjbGFyZS1mdW5jdGlv biBjaWRlci1jdXJyZW50LW5zICJleHQ6Y2lkZXItY2xpZW50IiAoKSkKIChkZWNsYXJlLWZ1bmN0 aW9uIG5yZXBsLS1tZXJnZSAiZXh0Om5yZXBsLWNsaWVudCIgKGRpY3QxIGRpY3QyKSkKIChkZWNs YXJlLWZ1bmN0aW9uIG5yZXBsLWRpY3QtZ2V0ICJleHQ6bnJlcGwtY2xpZW50IiAoZGljdCBrZXkp KQogKGRlY2xhcmUtZnVuY3Rpb24gbnJlcGwtZGljdC1wdXQgImV4dDpucmVwbC1jbGllbnQiIChk aWN0IGtleSB2YWx1ZSkpCkBAIC0xMTgsNyArMTE5LDcgQEAgdXNpbmcgdGhlIDpzaG93LXByb2Nl c3MgcGFyYW1ldGVyLiIKIAkJCSAgICBvcmctYmFiZWwtY2xvanVyZS1zeW5jLW5yZXBsLXRpbWVv dXQpKQogCQkgICAgICAgKG5yZXBsLXN5bmMtcmVxdWVzdDpldmFsIGV4cGFuZGVkCiAJCQkJCQko Y2lkZXItY3VycmVudC1jb25uZWN0aW9uKQotCQkJCQkJKGNpZGVyLWN1cnJlbnQtc2Vzc2lvbikp KSkKKwkJCQkJCShjaWRlci1jdXJyZW50LW5zKSkpKQogCSAgICAgICAoc2V0cSByZXN1bHQKIAkJ ICAgICAoY29uY2F0CiAJCSAgICAgIChucmVwbC1kaWN0LWdldCByZXNwb25zZQpAQCAtMTUzLDcg KzE1NCw3IEBAIHVzaW5nIHRoZSA6c2hvdy1wcm9jZXNzIHBhcmFtZXRlci4iCiAJCTs7IFVwZGF0 ZSB0aGUgc3RhdHVzIG9mIHRoZSBuUkVQTCBvdXRwdXQgc2Vzc2lvbi4KIAkJKHNldHEgc3RhdHVz IChucmVwbC1kaWN0LWdldCByZXNwb25zZSAic3RhdHVzIikpKQogCSAgICAgIChjaWRlci1jdXJy ZW50LWNvbm5lY3Rpb24pCi0JICAgICAgKGNpZGVyLWN1cnJlbnQtc2Vzc2lvbikpCisJICAgICAg KGNpZGVyLWN1cnJlbnQtbnMpKQogCiAJICAgICA7OyBXYWl0IHVudGlsIHRoZSBuUkVQTCBjb2Rl IGZpbmlzaGVkIHRvIGJlIHByb2Nlc3NlZC4KIAkgICAgICh3aGlsZSAobm90IChtZW1iZXIgImRv bmUiIHN0YXR1cykpCi0tIAoyLjcuNAoK --001a11418316d7022a0549cbee64--