Re: [PATCH] Add '--drop-all-empty-revs' to svndumpfilter include/exclude

On 02/26/2013 10:08 AM, vijay wrote:
> If the dump was taken from a mirror repo, r0 might have some information
> like 'svn:sync-from-url', 'svn:sync-from-uuid', etc.
> I thought that '--drop-all-empty-revs' should not just drop r0 considering
> as an empty revision.

That's seems fair. The load logic already special-cases revision 0, never
treating it as a "real" new revision to be created in the target repository.
 It's as consistent to special-case revision 0 here, too, as it would be to
not do so. But the usage message and docs should reveal this nuance, in my
opinion.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development