Re: [PATCH] Re: SVN & Diff & White Spaces
On Sun, 29 Jul 2007, Erik Huelsmann wrote:
> On 7/27/07, Stefan Sperling <email@example.com> wrote:
> > On Fri, Jul 27, 2007 at 02:58:40PM +0300, tom wrote:
> > > Sorry, have no chance atm.
> > > We've 1.3.2 installed, going move to 1.5 after release.
> > Well, I don't expect you to test this on a production
> > system of course :)
> > But if you have no time to test, that's OK.
> > @list: Should I create an issue for this so it does not get
> > lost in the mailing list archives?
> No need, I had it marked for review in my Inbox. I must say, the patch
> is very nice overall, but I have one nit. And it would be great if you
> wanted to address it: the svn command takes its diff arguments as the
> parameter of the -x option. It would be great if you could make it
> happen for svnlook too, because of consistency across the toolset.
+1 and +1.
Received on Mon Jul 30 17:53:22 2007
- application/pgp-signature attachment: stored
This is an archived mail posted to the Subversion Dev