On Feb 13, 2010, at 6:22 PM, Phil Hagelberg wrote: > On Sat, Feb 13, 2010 at 11:03 AM, Richard Newman <holyg...@gmail.com> wrote: >>> The above thread suggests defining *err* as a PrintWriter instead of >>> as a Writer. Has this been patched, and is it official? If so, I'll >>> patch clojure-swank to use PrintWriter. If not, I'll patch >>> clojure.contrib.sql to not use println. >> >> I patched swank-clojure: >> >> diff --git a/src/swank/core/server.clj b/src/swank/core/server.clj >> index 31c25ee..972e74a 100644 >> --- a/src/swank/core/server.clj >> +++ b/src/swank/core/server.clj >> >> It would be nice to see this upstream. > > Agreed! My patch backlog for swank-clojure is far bigger than I feel > comfortable with, but I've been focusing my attention on Leiningen for > the 1.1.0 release. Once that gets out I will turn to swank-clojure. > > I should mention that I could use some help co-maintaining > swank-clojure. I don't feel like I have a great grasp on the codebase > right now and have just been keeping it on autopilot after figuring > out packaging issues, but having someone who's familiar with how it > works from CL would be very helpful. Even without CL experience, just > gathering up the various patches into one branch and seeing what works > and what doesn't would be very helpful too. Otherwise I'll get to it > when I have time, but it might not be very prompt. > > -Phil
FWIW, Richard's patch is the simplest thing that could and does possibly work. +1. Thanks, Johnny -- 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 Note that posts from new members are moderated - please be patient with your first post. 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
