On Sat, 14 Jan 2006, Daniel Rall wrote:
> On Sat, 14 Jan 2006, Julian Foad wrote:
>
> > Daniel, if this is work in progress, you'd do much better to send it to the
> > mailing list where people could easily review it. There's no point in
> > putting it in the issue tracker unless you think the patch is going to have
> > to sit around for a long time before anyone looks at it. Unless I've
> > missed the point.
>
> Unless I missed it, there was no comment other than lundblad's IRC
> review last time I sent a version of this patch to the dev list.
>
I also find it easier (and more likely:-) to review patches sent to dev@
and it is easier to reply with comments inline as well.
> Personally, I generally do review patches attached to issues (at least
> internally at CollabNet and for the ASF), but if you prefer I continue
> to send revised versions to the dev list, I don't mind doing so.
> Kinda figured that I was on my own with this one...
>
Um? You're not on your own. I reviewed and S.Ramaswamy posted another
patch with some good stuff in it. Did you look at it? (I haven't looked
at your latest patch, so I don't know.)
Thanks,
//Peter
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Jan 14 22:21:40 2006