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

Re: [PATCH] Issue 1074 (Take 3)

From: John Szakmeister <john_at_szakmeister.net>
Date: 2003-08-25 12:29:31 CEST

On Sunday 24 August 2003 19:31, cmpilato@collab.net wrote:
> John Szakmeister <john@szakmeister.net> writes:
> > Here is the next attempt at a patch for Issue 1074 (Need a svnadmin
> > command to verify that the repository is not corrupted).
> >
> > There were a couple of things I was unsure about. I went ahead and added
> > subcommand_verify to svnadmin, but it really calls subcommand_dump after
> > setting a couple of options.
>
> Make sense. Perhaps a cleaner way would just be to core the guts out
> of subcommand_dump into a helper function that is called by both
> subcommand_dump and subcommand_verify.

Yeah, I don't know why I didn't do that in the first place.

> > It sets one new one called 'verify' so that the dump command knows
> > to create a NULL dumpstream. Is it okay to have this 'verify' option
> > in opt_state, or should I find a different way to do this?
>
> Oh, no, please don't add options to opt_state that don't reflect real
> commandline options. See the above suggestion.

I'll fix this up, and re-submit the patch.

> > I also set the incremental option. Since we're walking through the
> > entire repository, I didn't feel it was necessary to have it create
> > the full-text at every revision. Any problems with doing this?
>
> Nope, that's all good, as long as this subcommand doesn't accept the
> -r option.

Nope, it doesn't.

> > C-Mike, I've cc'd you on this because you seem to be my patch
> > reviewer/mentor for this issue. :-)
>
> I'll try to review the patch itself (instead of just your comments
> about it above) Real Soon. Thanks, John.

Cool. I'll try and get the new patch out later tonight after I get off work.

Thanks,
John

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Aug 25 12:29:55 2003

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.