[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: Testing equality between svnrdump and svnadmin dump

From: Bert Huijben <bert_at_qqmail.nl>
Date: Tue, 13 Jan 2015 18:08:41 +0100

> -----Original Message-----
> From: Julian Foad [mailto:julianfoad_at_btopenworld.com]
> Sent: dinsdag 13 januari 2015 15:22
> To: Branko Čibej
> Cc: dev_at_subversion.apache.org
> Subject: Re: Testing equality between svnrdump and svnadmin dump
>
> The next version of my testing patch is attached. It pipes each dumpfile through
> svndumpfilter and checks that a no-op filtering does not change anything.

I haven't reviewed all of this, but I'm wondering why in your patch
[[
   def parse_path(self):
- path = self.parse_line('Node-path: (.+)$', required=False)
- if not path and self.lines[self.current] == 'Node-path: \n':
- self.current += 1
- path = ''
+ path = self.parse_line('Node-path: (.*)$', required=False)
]]

This replaces '.+' with '.*'

I was wondering why this was necessary?
(To catch more error? Or does this skip some errors?)

Nice catches in the other mails on this subject!

Thanks,
        Bert
Received on 2015-01-13 18:10:01 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.