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 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 100644 > --- a/swank/commands/basic.clj > +++ b/swank/commands/basic.clj > @@ -273,7 +273,7 @@ > (when-let [meta (and sym-var (meta sym-var))] > (if-let [path (or (slime-find-file-in-paths (:file meta) (slime- > search-paths)) > (slime-find-file-in-paths (str (namespace-to- > path (:ns meta)) > - (.separator > File) > + File/separator > (:file meta)) > (slime-search-paths)) > `((~(str "(defn " (:name meta) ")") > (:location > diff --git a/swank/util/io.clj b/swank/util/io.clj > index 56b037e..48fd5b8 100644 > --- a/swank/util/io.clj > +++ b/swank/util/io.clj > @@ -28,7 +28,7 @@ > (dothread > (thread-set-name "Call-on-write Stream") > (continuously > - (.sleep Thread 200) > + (Thread/sleep 200) > (when-not @closed? > (.flush stream)))) > > > > > On Dec 14, 4:41 pm, kunnar <kunnar.kla...@gmail.com> wrote: >> 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 - clojure itself, swank-clojure, >> clojure-mode, slime, clojure-contrib, but i still get that error. >> Before update "M-." did work. >> >> Kunnar >> >> On Dec 11, 7:04 am, "Bill Clementson" <billc...@gmail.com> wrote: >> >> > 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 <billc...@gmail.com> >> > wrote: >> > > Hi Matt, >> >> > > On Wed, Dec 10, 2008 at 12:47 PM, MattyDub <mattydu...@gmail.com> 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 doesslime-edit-definitionsearch 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 >> >> > > Basically, it searches a bunch of standard Java locations plus the >> > > locations you specified in your classpath. If you want to see what >> > > will be searched, enter the following 2 statements in your Clojure >> > > REPL: >> > > (in-ns 'swank.commands.basic) >> > > (slime-search-paths) >> >> > > Note: the function "slime-search-paths" isn't public >> >> > > If M-. isn't working on standard Clojure symbols, it could be for a >> > > number of reasons: >> > > 1. You're using an old version of Clojure >> > > 2. You've built Clojure excluding source from the jar file >> > > 3. The Clojure source isn't in your classpath (only necessary if #2 is >> > > true) >> >> > > - Bill > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this group, send email to clojure@googlegroups.com To unsubscribe from this group, send email to clojure+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/clojure?hl=en -~----------~----~----~----~------~----~------~--~---
