Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-16 Thread Bill Clementson
FYI - I forwarded the patch to Jeffrey Chu and he has patched swank-clojure in git. - Bill On Sun, Dec 14, 2008 at 2:20 PM, Feng hou...@gmail.com wrote: This is due to the change in clojure svn 1158. (.method ClassName ...) syntax has been changed to call java.lang.Class method for

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-14 Thread kunnar
Hi, Looks like patch is already applied, but i now i get following error after M-. java.lang.IllegalArgumentException: No matching field found: separator for class java.lang.Class (NO_SOURCE_FILE:0) [Thrown class clojure.lang.Compiler$CompilerException] I updated everything clojure related

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-14 Thread Feng
This is due to the change in clojure svn 1158. (.method ClassName ...) syntax has been changed to call java.lang.Class method for ClassName, not static field/method any more. Use namespace syntax instead. diff --git a/swank/commands/basic.clj b/swank/commands/basic.clj index ac0736f..4705cd5

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-11 Thread MattyDub
Well, I had the .clj files in my clojure.jar, and clojure.jar is on slime-search-paths, but I still wasn't getting M-. to work with clojure's functions and macros. I added the source files to my classpath, and I get them now. Also, I updated clojure and swank-clojure this morning, so it looks

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-10 Thread MattyDub
Thanks for confirming the bug, Bill. Should we post this to Jeffrey Chu (assuming it's a swank-clojure issue)? What's the protocol at this point? -Matt On Dec 9, 11:03 pm, Bill Clementson [EMAIL PROTECTED] wrote: Hi Matt, On Tue, Dec 9, 2008 at 6:43 PM, MattyDub [EMAIL PROTECTED] wrote:

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-10 Thread Bill Clementson
Hi Matt, Yes, you should let Jeffrey know about the bug. - Bill On Wed, Dec 10, 2008 at 9:20 AM, MattyDub [EMAIL PROTECTED] wrote: Thanks for confirming the bug, Bill. Should we post this to Jeffrey Chu (assuming it's a swank-clojure issue)? What's the protocol at this point? -Matt On

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-10 Thread Bill Clementson
Hi Matt, I had a bit of free time this morning, so I looked into this further and came up with a patch (attached). I've forwarded the patch to Jeffrey as well, so it should find it's way into swank-clojure in due course. Cheers, Bill Clementson On Wed, Dec 10, 2008 at 9:57 AM, Bill Clementson

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-10 Thread Bill Clementson
Oops, that patch had my debugging statements in it. Here's the correct one to use. On Wed, Dec 10, 2008 at 11:44 AM, Bill Clementson [EMAIL PROTECTED] wrote: Hi Matt, I had a bit of free time this morning, so I looked into this further and came up with a patch (attached). I've forwarded the

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-10 Thread Bill Clementson
Hi Matt, On Wed, Dec 10, 2008 at 12:47 PM, MattyDub [EMAIL PROTECTED] wrote: I can confirm that that patch fixed my problem - I can now M-. to render-place. Thanks, Bill! Good to hear that the patch fixes the issue for you. What paths does slime-edit-definition search for the

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-10 Thread Bill Clementson
Hi Matt, FYI - Jeffrey Chu just sent me an email and the patch has now been applied to swank-clojure. - Bill On Wed, Dec 10, 2008 at 1:09 PM, Bill Clementson [EMAIL PROTECTED] wrote: Hi Matt, On Wed, Dec 10, 2008 at 12:47 PM, MattyDub [EMAIL PROTECTED] wrote: I can confirm that that patch

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-09 Thread MattyDub
I recently tried M-. for the first time with clojure + SLIME and it broke for me (with a Lisp error: (error Synchronous Lisp Evaluation aborted.)); other parts of SLIME work (e.g., C-M-x, or C-x C-e), but this doesn't . I didn't understand the previous messages in this thread: 1) Stephen Gilardi

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-09 Thread Bill Clementson
Hi Matt, On Tue, Dec 9, 2008 at 4:15 PM, MattyDub [EMAIL PROTECTED] wrote: I recently tried M-. for the first time with clojure + SLIME and it broke for me (with a Lisp error: (error Synchronous Lisp Evaluation aborted.)); other parts of SLIME work (e.g., C-M-x, or C-x C-e), but this

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-09 Thread MattyDub
I did an 'svn update' on my clojure (I'm now at revision 1149), and ran ant clean, then ant (the default target). Then I deleted my old swank-clojure directory, and did a git clone of git://github.com/jochu/swank-clojure.git. I made sure my .emacs was correctly pointing to the appropriate

Re: Patch: precompiling Clojure core sources in Ant build script

2008-12-09 Thread Bill Clementson
Hi Matt, On Tue, Dec 9, 2008 at 6:43 PM, MattyDub [EMAIL PROTECTED] wrote: I did an 'svn update' on my clojure (I'm now at revision 1149), and ran ant clean, then ant (the default target). Then I deleted my old swank-clojure directory, and did a git clone of

Re: Patch: precompiling Clojure core sources in Ant build script

2008-11-14 Thread Rich Hickey
On Nov 14, 12:51 pm, Stuart Sierra [EMAIL PROTECTED] wrote: Patch attached. This precompiles everything except parallel.clj, which requires an extra Jar. It also omits the source .clj files from the clojure.jar file. -Stuart Sierra Patch applied (svn 1101) - thanks! Rich

Re: Patch: precompiling Clojure core sources in Ant build script

2008-11-14 Thread Graham Fawcett
On Fri, Nov 14, 2008 at 2:37 PM, Rich Hickey [EMAIL PROTECTED] wrote: On Nov 14, 12:51 pm, Stuart Sierra [EMAIL PROTECTED] wrote: Patch attached. This precompiles everything except parallel.clj, which requires an extra Jar. It also omits the source .clj files from the clojure.jar file.

Re: Patch: precompiling Clojure core sources in Ant build script

2008-11-14 Thread Stephen C. Gilardi
On Nov 14, 2008, at 3:38 PM, Graham Fawcett wrote: I'm sure there's a work-around, of course. The meta information is still there: This appears to be because the sources are no longer included in clojure.jar. I added path-to-clojure-trunk-svn-directory/src/clj to my classpath and now it

Re: Patch: precompiling Clojure core sources in Ant build script

2008-11-14 Thread Graham Fawcett
On Fri, Nov 14, 2008 at 4:01 PM, Stephen C. Gilardi [EMAIL PROTECTED] wrote: On Nov 14, 2008, at 3:38 PM, Graham Fawcett wrote: I'm sure there's a work-around, of course. The meta information is still there: This appears to be because the sources are no longer included in clojure.jar. I

Re: Patch: precompiling Clojure core sources in Ant build script

2008-11-14 Thread Feng
On Nov 14, 4:09 pm, Graham Fawcett [EMAIL PROTECTED] wrote: On Fri, Nov 14, 2008 at 4:01 PM, Stephen C. Gilardi [EMAIL PROTECTED] wrote: On Nov 14, 2008, at 3:38 PM, Graham Fawcett wrote: I'm sure there's a work-around, of course. The meta information is still there: This appears