[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-dev
Subject:    Re: [PATCH] Re: SVN & Diff & White Spaces
From:       Stefan Sperling <stsp () elego ! de>
Date:       2007-07-30 16:24:52
Message-ID: 20070730162452.GA2546 () ted ! stsp ! lan
[Download RAW message or body]


On Mon, Jul 30, 2007 at 08:53:29AM -0700, Daniel Rall wrote:
> On Sun, 29 Jul 2007, Erik Huelsmann wrote:
> 
> > On 7/27/07, Stefan Sperling <stsp@elego.de> 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.

As already mentioned in a private mail to Erik I'm currently on holiday.
So I won't be able to look into the -x parameter until the end
of next week.

See you then :)
-- 
Stefan Sperling <stsp@elego.de>                 Software Developer
elego Software Solutions GmbH                            HRB 77719
Gustav-Meyer-Allee 25, Gebaeude 12        Tel:  +49 30 23 45 86 96 
13355 Berlin                              Fax:  +49 30 23 45 86 95
http://www.elego.de                 Geschaeftsfuehrer: Olaf Wagner

[Attachment #3 (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic