Re: [PATCH] Implement svnadmin verify --force

2012-10-27 Thread Daniel Shahaf
> Index: subversion/libsvn_fs/fs-loader.c > === > --- subversion/libsvn_fs/fs-loader.c (revision 1402414) > +++ subversion/libsvn_fs/fs-loader.c (working copy) > @@ -487,17 +487,21 @@ >void *cancel_baton, >

Re: [PATCH] Implement svnadmin verify --force

2012-10-27 Thread Prabhu Gnana Sundar
Thank you so much Stefan and Daniel. I learnt so much from you guys during the course of this patch. I attached a new patch and a log message with this mail. Please let me know if I missed out on something. Thanks and regards Prabhu On 10/27/2012 09:59 PM, Daniel Shahaf wrote: Stefan Spe

Re: [PATCH] Implement svnadmin verify --force

2012-10-27 Thread Daniel Shahaf
Stefan Sperling wrote on Sat, Oct 27, 2012 at 10:46:59 +0200: > I don't think this is a problem at all. If revision A has a broken > rep and revision B refers to that rep, then both A and B are broken. > Even if the source of the corruption is in A only I want to know that > B is affected by it. Be

Re: [PATCH] Implement svnadmin verify --force

2012-10-27 Thread Stefan Sperling
On Fri, Oct 26, 2012 at 07:07:45PM +0200, Daniel Shahaf wrote: > Stefan Sperling wrote on Fri, Oct 26, 2012 at 12:45:49 +0200: > > I'd like to have this feature because it will make my work a little > > easier should I ever be tasked again with repairing broken repositories. > > When I last had to

Re: 'svn mergeinfo' graph bikeshed patch

2012-10-27 Thread Branko Čibej
On 27.10.2012 07:25, Peter Samuelson wrote: > [Branko Čibej] >> On 26.10.2012 21:04, C. Michael Pilato wrote: >>> Just felt like painting. >> The only problem I see with this is that, in many monospaced fonts, ' >> and ` glyphs are not mirror images; whereas / and \ almost always are. > Well, note