[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [patch] ob-clojure: Fix results output
From: |
Daniel Kraus |
Subject: |
Re: [patch] ob-clojure: Fix results output |
Date: |
Wed, 15 Mar 2023 12:22:17 +0100 |
Ihor Radchenko <yantar92@posteo.net> writes:
> What will happen with users who customized `org-babel-clojure-backend'
> to 'nbb in the past?
They would have gotten an error.
I changed it now that 'nbb backend is still allowed in a clojure
source block but it will internally treated as ClojureScript.
>> +(defcustom org-babel-clojurescript-backend
>> + (cond
>> + ((or (executable-find "nbb") (executable-find "npx")) 'nbb)
>> + ((featurep 'cider) 'cider)
>> + (t nil))
>> + "Backend used to evaluate Clojure code blocks."
>
> This docstring is exactly the same with `org-babel-clojure-backend'.
> What is the difference?
> I think ""Backend used to evaluate ClojureScript code blocks." will be
> more clear. I feel that other docstrings may also need to be clarified
> depending whether they affect Clojure or ClojureScript blocks.
I changed the docstrings to always mention either Clojure or ClojureScript.
I'm open for more improvements/suggestions.
Attached a new patch.
Thanks,
Daniel
0001-ob-clojure.el-Fix-results-output-support-clojure-cli.patch
Description: Text Data
- [patch] ob-clojure: Fix results output, Daniel Kraus, 2023/03/09
- Re: [patch] ob-clojure: Fix results output, Ihor Radchenko, 2023/03/10
- Re: [patch] ob-clojure: Fix results output, Daniel Kraus, 2023/03/13
- Re: [patch] ob-clojure: Fix results output, Ihor Radchenko, 2023/03/14
- Re: [patch] ob-clojure: Fix results output, Daniel Kraus, 2023/03/14
- Re: [patch] ob-clojure: Fix results output, Daniel Kraus, 2023/03/14
- Re: [patch] ob-clojure: Fix results output, Ihor Radchenko, 2023/03/15
- Re: [patch] ob-clojure: Fix results output,
Daniel Kraus <=
- Re: [patch] ob-clojure: Fix results output, Ihor Radchenko, 2023/03/16
- Re: [patch] ob-clojure: Fix results output, Ihor Radchenko, 2023/03/19
- Re: [patch] ob-clojure: Fix results output, Daniel Kraus, 2023/03/19
- Re: [patch] ob-clojure: Fix results output, Ihor Radchenko, 2023/03/22
- Re: [patch] ob-clojure: Fix results output, Daniel Kraus, 2023/03/23
- Re: [patch] ob-clojure: Fix results output, Ihor Radchenko, 2023/03/23
- Re: [patch] ob-clojure: Fix results output, Daniel Kraus, 2023/03/23