* ob-clojure.el is not compatible with new CIDER API
@ 2018-07-16 11:17 stardiviner
2018-07-18 22:30 ` Tim Cross
0 siblings, 1 reply; 7+ messages in thread
From: stardiviner @ 2018-07-16 11:17 UTC (permalink / raw)
To: org-mode
I found ob-clojure.el is not compatible with new CIDER API now.
I don't know CIDER much, is there anybody want to update it?
--
[ stardiviner ] don't need to convince with trends.
Blog: https://stardiviner.github.io/
IRC(freenode): stardiviner
GPG: F09F650D7D674819892591401B5DF1C95AE89AC3
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ob-clojure.el is not compatible with new CIDER API
2018-07-16 11:17 ob-clojure.el is not compatible with new CIDER API stardiviner
@ 2018-07-18 22:30 ` Tim Cross
2018-07-24 4:32 ` stardiviner
0 siblings, 1 reply; 7+ messages in thread
From: Tim Cross @ 2018-07-18 22:30 UTC (permalink / raw)
To: numbchild; +Cc: org-mode
I did this a couple of times in the past, but not long afterwards, CIDER
would again change and the interface would no longer work.
Personally, I'm not convinced that using CIDER actually provides much
advantage at this time due to how rapidly it is evolving. Org would
likely do better just having an interface based around a simple
interface, such a inf-clojure.
stardiviner <numbchild@gmail.com> writes:
> I found ob-clojure.el is not compatible with new CIDER API now.
> I don't know CIDER much, is there anybody want to update it?
--
Tim Cross
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ob-clojure.el is not compatible with new CIDER API
2018-07-18 22:30 ` Tim Cross
@ 2018-07-24 4:32 ` stardiviner
2018-07-24 6:54 ` Tim Cross
0 siblings, 1 reply; 7+ messages in thread
From: stardiviner @ 2018-07-24 4:32 UTC (permalink / raw)
To: Tim Cross; +Cc: org-mode
Tim Cross <theophilusx@gmail.com> writes:
> I did this a couple of times in the past, but not long afterwards, CIDER
> would again change and the interface would no longer work.
>
> Personally, I'm not convinced that using CIDER actually provides much
> advantage at this time due to how rapidly it is evolving. Org would
> likely do better just having an interface based around a simple
> interface, such a inf-clojure.
>
Hi, Tim, I checked "inf-clojure", you're right, ob-clojure should migrate to inf-clojure. it is more stable and more suitable for ob-clojure. Have you tried this before?
--
[ stardiviner ] don't need to convince with trends.
Blog: https://stardiviner.github.io/
IRC(freenode): stardiviner
GPG: F09F650D7D674819892591401B5DF1C95AE89AC3
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ob-clojure.el is not compatible with new CIDER API
2018-07-24 4:32 ` stardiviner
@ 2018-07-24 6:54 ` Tim Cross
2018-07-29 11:59 ` stardiviner
0 siblings, 1 reply; 7+ messages in thread
From: Tim Cross @ 2018-07-24 6:54 UTC (permalink / raw)
To: 路Ricardo M.; +Cc: Org-mode
[-- Attachment #1: Type: text/plain, Size: 1270 bytes --]
I did begin to look at it. From memory, the inf-clojure integration didn't
look that hard, but it did require some additional scaffolding to mae the
comms work well. I then got distracted with a new job and haven't had time
to go back to it. Currently, I've not had time to do any Clojure work, so
it is not high on the priority list.
On Tue, 24 Jul 2018 at 15:09, stardiviner <numbchild@gmail.com> wrote:
>
> Tim Cross <theophilusx@gmail.com> writes:
>
> > I did this a couple of times in the past, but not long afterwards, CIDER
> > would again change and the interface would no longer work.
> >
> > Personally, I'm not convinced that using CIDER actually provides much
> > advantage at this time due to how rapidly it is evolving. Org would
> > likely do better just having an interface based around a simple
> > interface, such a inf-clojure.
> >
>
> Hi, Tim, I checked "inf-clojure", you're right, ob-clojure should migrate
> to inf-clojure. it is more stable and more suitable for ob-clojure. Have
> you tried this before?
>
> --
> [ stardiviner ] don't need to convince with trends.
> Blog: https://stardiviner.github.io/
> IRC(freenode): stardiviner
> GPG: F09F650D7D674819892591401B5DF1C95AE89AC3
>
>
>
--
regards,
Tim
--
Tim Cross
[-- Attachment #2: Type: text/html, Size: 2023 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ob-clojure.el is not compatible with new CIDER API
2018-07-24 6:54 ` Tim Cross
@ 2018-07-29 11:59 ` stardiviner
2018-07-29 22:14 ` Tim Cross
0 siblings, 1 reply; 7+ messages in thread
From: stardiviner @ 2018-07-29 11:59 UTC (permalink / raw)
To: Tim Cross; +Cc: Org-mode
Tim Cross <theophilusx@gmail.com> writes:
> I did begin to look at it. From memory, the inf-clojure integration didn't look that hard,
> but it did require some additional scaffolding to mae the comms work well. I then got
> distracted with a new job and haven't had time to go back to it. Currently, I've not had time
> to do any Clojure work, so it is not high on the priority list.
>
Can I see your work? Do you have any public source code repository online of this?
--
[ stardiviner ] don't need to convince with trends.
Blog: https://stardiviner.github.io/
IRC(freenode): stardiviner
GPG: F09F650D7D674819892591401B5DF1C95AE89AC3
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ob-clojure.el is not compatible with new CIDER API
2018-07-29 11:59 ` stardiviner
@ 2018-07-29 22:14 ` Tim Cross
2018-07-31 0:25 ` stardiviner
0 siblings, 1 reply; 7+ messages in thread
From: Tim Cross @ 2018-07-29 22:14 UTC (permalink / raw)
To: numbchild; +Cc: Org-mode
No, there is no public repo and I'm not sure I even kept what little I
had done - it was just a hacked elisp file. I'll have a look through my
elisp archive, but don't expect much.
Over the weekend, I caught up on a bit of clojure and things have
progressed in the last few months. I notice the clj tool seems quite
mature and there is now support for clojurescript with it as well. Don't
know if inf-clojure will move (has already moved?) to be based around it
or not?
At any rate, I think you are probably better off starting from scratch
rather than trying to 'adjust' the existing package. There are some
cases where I can see someone wanting cider/slime support, but for the
majority of cases, just plain clojure via inf-clojure or clojure/clj or
one of the many available repl clients should be fairly straight
forward.
The real challenge is in deciding how to handle interaction between
different clojure blocks and their dependencies. The simple solution
would likely be to just start some form of repl session and have all
blocks sent to that session. However, you then need to decide how to
handle matters when someone opens a new org file. You probably don't
want old definitions/artifacts from the previous session in the repl, so
you would need a clean way of managing multiple sessions (one per org
file?).
Apart from that, all you really need is the ability to send a form to
the repl and read back the result. The inf-clojure package already has
most of the infrastructure for doing that, but from memory, it was a
little hard to fit it into the existing structure of ob-clojure.el -
which is why you may be better off just implementing an
ob-inf-clojure.el rather than try to add a 3rd method to the existing
package (which supports cide and the old slime-clojure interfaces).
stardiviner <numbchild@gmail.com> writes:
> Tim Cross <theophilusx@gmail.com> writes:
>
>> I did begin to look at it. From memory, the inf-clojure integration didn't look that hard,
>> but it did require some additional scaffolding to mae the comms work well. I then got
>> distracted with a new job and haven't had time to go back to it. Currently, I've not had time
>> to do any Clojure work, so it is not high on the priority list.
>>
>
> Can I see your work? Do you have any public source code repository online of this?
--
Tim Cross
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ob-clojure.el is not compatible with new CIDER API
2018-07-29 22:14 ` Tim Cross
@ 2018-07-31 0:25 ` stardiviner
0 siblings, 0 replies; 7+ messages in thread
From: stardiviner @ 2018-07-31 0:25 UTC (permalink / raw)
To: Tim Cross; +Cc: Org-mode
About 3 days ago, I updated CIDER, seems it can work again now. After considering your mentioned problems of managing sessions, seems CIDER implemented new middleware called "sesman", have not tried it in details, I will dig into it later.
Hmm, because I use ob-clojure to do "Literate Programming", so if can't support it well, I will not consider to do it. So to implement one "ob-inf-clojure.el" is kind of expensive for me.
--
[ stardiviner ] don't need to convince with trends.
Blog: https://stardiviner.github.io/
IRC(freenode): stardiviner
GPG: F09F650D7D674819892591401B5DF1C95AE89AC3
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-07-31 0:25 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-16 11:17 ob-clojure.el is not compatible with new CIDER API stardiviner
2018-07-18 22:30 ` Tim Cross
2018-07-24 4:32 ` stardiviner
2018-07-24 6:54 ` Tim Cross
2018-07-29 11:59 ` stardiviner
2018-07-29 22:14 ` Tim Cross
2018-07-31 0:25 ` stardiviner
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.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).