Thanks, Stephen -- good feedback! I fixed [2] and [3]. Not sure about [1], maybe it's a line-ending thing. Anyway, this new patch (attached) was written on OSX, so it had better work! -S
On Nov 21, 5:31 pm, "Stephen C. Gilardi" <[EMAIL PROTECTED]> wrote: > On Nov 21, 2008, at 12:34 PM, Stuart Sierra wrote: > > > The attached patch combines Repl, Script, and the lib compiler that > > Stephen G. and I have worked on. > > Hi Stuart, > > I like the concept very much. > > I had a couple of problems in working with the patch: > > [1] patch on Mac OS X Leopard rejected the build.xml portion of the > patch you posted. I'm not sure what the problem was. As a result, I > made the changes manually to a fresh svn 1121 build.xml. > > [2] your clojure.lang.Main uses Arrays.copyOfRange which is apparently > unavailable until Java SE 6. We are targeting Java 1.5 in the ant > task. I'm unable to get this to compile even if I remove the > "target=1.5" option. I find that a bit odd because my default java > tools should be Java 6-based. > > [3] Is the example at the top of build.xml under "description" still > correct? I would like to see the example be "java -jar clojure.jar" to > launch a REPL. > > Thanks for Main! I think it's a great idea and I look forward to using > it. > > --Steve --~--~---------~--~----~------------~-------~--~----~ 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 [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/clojure?hl=en -~----------~----~----~----~------~----~------~--~---
universal-main2.patch
Description: Binary data
