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 wrote: > > 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

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 1006

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-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 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

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 defin

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

2008-12-10 Thread MattyDub
I can confirm that that patch fixed my problem - I can now M-. to render-place. Thanks, Bill! What paths does slime-edit-definition search for the definitions? It's not finding the clojure macros/functions (like defn, dosync, etc.), so I must not have that path configured correctly. -Matt On

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

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
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? > -Mat

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]

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 > git://github.com/jochu/s

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 paths.

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 > t

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 Gila

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:

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

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 "/src/clj" to my classpath and now it works. --Steve --~--~---

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 cl

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 --~--~

Patch: precompiling Clojure core sources in Ant build script

2008-11-14 Thread Stuart Sierra
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 --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Gro