Re: [O] update to ob-clojure.el

2014-04-17 Thread Bastien
Hi Greg and Oleh,

Soapy Smith soapy-sm...@comcast.net writes:

 I'm not sure about removing nrepl and slime, as there may be some still
 using them, especially nrepl.  As long as it is not getting in the way,
 perhaps at this time it would be better to leave them in?

Agreed.

 I'm not sure
 the open-source and/or emacs-org philosophy on this kind of thing.

If it ain't broken, don't fix it.

 Prior to updating the documentation for Clojure code blocks earlier this
 year, which was several years old, I did a lot of searching around and
 came to the conclusion that the slime and swank-clojure functionality
 was no longer being developed, and the community had moved on to nrepl.
 CIDER evolved from nrepl, and is the current direction.

 https://github.com/clojure-emacs/cider

 The revised documentation is very CIDER-centric:

 http://orgmode.org/worg/org-contrib/babel/languages/ob-doc-clojure.html

Yes, we shall put all efforts into supporting CIDER, and then ping the
Clojure community to know whether slime and nrepl still need to be
supported.  But for Org 8.3, let's continue to have those.

-- 
 Bastien



Re: [O] update to ob-clojure.el

2014-04-14 Thread Oleh
 I've made an update to ob-clojure.el, which wasn't working for me with
 the new cider (the required function was removed).

 I haven't looked into 'nrepl or 'slime options for evaluation, is
 anyone using them?

 Eric, should 'nrepl option be removed as obsolete? And maybe 'slime as well?


 I'm not sure about removing nrepl and slime, as there may be some still
 using them, especially nrepl.  As long as it is not getting in the way,
 perhaps at this time it would be better to leave them in?  I'm not sure
 the open-source and/or emacs-org philosophy on this kind of thing.

 Prior to updating the documentation for Clojure code blocks earlier this
 year, which was several years old, I did a lot of searching around and
 came to the conclusion that the slime and swank-clojure functionality
 was no longer being developed, and the community had moved on to nrepl.
 CIDER evolved from nrepl, and is the current direction.

 https://github.com/clojure-emacs/cider

 The revised documentation is very CIDER-centric:

 http://orgmode.org/worg/org-contrib/babel/languages/ob-doc-clojure.html

My point was that if even CIDER, which is a favorite method, has
bit-rotted to the point of not working, makes it a bit questionable to
try to support the other two methods.

For instance, I could not get SLIME to work with org when I tried a
year ago, so I don't know if it's working now.

And CIDER replaces nREPL, so nREPL should be dropped at some point.
On the other hand, since nREPL isn't being edited now it's unlikely to
produce the same problem as CIDER did (the function used by ob-clojure
was removed from CIDER).

regards,
Oleh



Re: [O] update to ob-clojure.el

2014-04-13 Thread Soapy Smith
On Thu, 2014-04-10 at 10:36 +0200, Oleh wrote:
 Hi all,
 
 I've made an update to ob-clojure.el, which wasn't working for me with
 the new cider (the required function was removed).
 
 I haven't looked into 'nrepl or 'slime options for evaluation, is
 anyone using them?
 
 Eric, should 'nrepl option be removed as obsolete? And maybe 'slime as well?
 
 regards,
 Oleh

I'm not sure about removing nrepl and slime, as there may be some still
using them, especially nrepl.  As long as it is not getting in the way,
perhaps at this time it would be better to leave them in?  I'm not sure
the open-source and/or emacs-org philosophy on this kind of thing.

Prior to updating the documentation for Clojure code blocks earlier this
year, which was several years old, I did a lot of searching around and
came to the conclusion that the slime and swank-clojure functionality
was no longer being developed, and the community had moved on to nrepl.
CIDER evolved from nrepl, and is the current direction.

https://github.com/clojure-emacs/cider

The revised documentation is very CIDER-centric:

http://orgmode.org/worg/org-contrib/babel/languages/ob-doc-clojure.html

Regards,
Greg




Re: [O] update to ob-clojure.el

2014-04-11 Thread Bastien
Hi Oleh,

Oleh ohwoeo...@gmail.com writes:

 As long as we have to support Emacs 23, I think you shouldn't replace
 case with cl-case.

 I've reverted it. I'll keep this in mind for future commits, thanks.

Thanks for the patch -- and since we are nitpicking, here is another
one (a famous one I'd say): use double-space for ending sentences in
the Changelog entry :)

-- 
 Bastien



[O] update to ob-clojure.el

2014-04-10 Thread Oleh
Hi all,

I've made an update to ob-clojure.el, which wasn't working for me with
the new cider (the required function was removed).

I haven't looked into 'nrepl or 'slime options for evaluation, is
anyone using them?

Eric, should 'nrepl option be removed as obsolete? And maybe 'slime as well?

regards,
Oleh



Re: [O] update to ob-clojure.el

2014-04-10 Thread Nicolas Goaziou
Hello,

Oleh ohwoeo...@gmail.com writes:

 I've made an update to ob-clojure.el, which wasn't working for me with
 the new cider (the required function was removed).

As long as we have to support Emacs 23, I think you shouldn't replace
case with cl-case.


Regards,

-- 
Nicolas Goaziou



Re: [O] update to ob-clojure.el

2014-04-10 Thread Oleh
Hi Nicolas,


 I've made an update to ob-clojure.el, which wasn't working for me with
 the new cider (the required function was removed).

 As long as we have to support Emacs 23, I think you shouldn't replace
 case with cl-case.

I've reverted it. I'll keep this in mind for future commits, thanks.

regards,
Oleh