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

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

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Tue, 26 Feb 2013 14:17:02 +0000

vijay <vijay_at_collab.net> writes:

> /* write out the revision */
> /* Revision is written out in the following cases:
> - 1. No --drop-empty-revs has been supplied.
> - 2. --drop-empty-revs has been supplied,
> - but revision has not all nodes dropped
> - 3. Revision had no nodes to begin with.
> + 1. If the revision has nodes or it is revision 0.
> + 2. --drop-empty-revs has been supplied,
> + but revision has not all nodes dropped.
> + 3. No --drop-all-empty-revs has been supplied.
> + 4. If no --drop-empty-revs or --drop-all-empty-revs have been supplied,
> + write out the revision which has no nodes to begin with.
> */
> - if (rb->has_nodes
> - || (! rb->pb->drop_empty_revs)
> - || (! rb->had_dropped_nodes))
> + if (rb->has_nodes || (rb->rev_orig == 0))
> + write_out_rev = TRUE;
> + else if (rb->pb->drop_empty_revs)
> + write_out_rev = rb->had_dropped_nodes ? FALSE : TRUE;
> + else if (rb->pb->drop_all_empty_revs)
> + write_out_rev = FALSE;
> + else
> + write_out_rev = TRUE;
> +
> + if (write_out_rev)
> {

You have added the rev_orig == 0 check so that means that
--drop-all-empty-revs doesn't drop r0. Is there a reason for that
behaviour?

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download
Received on 2013-02-26 15:17:54 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.