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

Re: SVN 1.5-beta1 svn mergeinfo output for deleted branches

From: Erik Grinaker <erikg_at_codepoet.no>
Date: Tue, 25 Mar 2008 17:33:32 +0700

On Tue, 2008-03-25 at 15:50 +0530, Bhuvaneswaran Arumugam wrote:
> On Tue, 2008-03-25 at 15:55 +0700, Erik Grinaker wrote:
> > On Tue, 2008-03-25 at 13:17 +0530, Bhuvaneswaran Arumugam wrote:
> > > On Tue, 2008-03-25 at 13:43 +0700, Erik Grinaker wrote:
> > > > On Mon, 2008-03-24 at 14:37 +0100, Karl Heinz Marbaise wrote:
> > > > > Hi there,
> > > > >
> > > > > i have a questions about the behaviour of svn mergeinfo...
> > > > >
> > > > > I have created branch, merged from that branch and so on....
> > > > >
> > > > > svn mergeinfo
> > > > > Path: .
> > > > > Source path: /project1/trunk
> > > > > Merged ranges: r1:21
> > > > > Eligible ranges: r31:34
> > > > > Source path: /project1/branches/B_ENH_F1
> > > > > Merged ranges: r2:33
> > > > > Eligible ranges:
> > > > > Source path: /project1/branches/B_DIFF
> > > > > Merged ranges: r21:23
> > > > > Eligible ranges: r23:33
> > > > >
> > > > > Ok so far so god...
> > > > >
> > > > > But than i deleted the branch B_ENH_F1, cause i completed the work on
> > > > > that branch...
> > > > >
> > > > > svn rm ../project1/branches/B_ENH_F1 -m"- Done."
> > > > >
> > > > > and now
> > > > > svn mergeinfo
> > > > > Path: .
> > > > > Source path: /project1/trunk
> > > > > Merged ranges: r1:21
> > > > > Eligible ranges: r31:34
> > > > > Source path: /project1/branches/B_ENH_F1
> > > > > Merged ranges: r2:33
> > > > > Eligible ranges: (source no longer available in HEAD)
> > > > > Source path: /project1/branches/B_DIFF
> > > > > Merged ranges: r21:23
> > > > > Eligible ranges: r23:33
> > > > >
> > > > > So is it no more best practice to remove the integrated branches or no
> > > > > more used branches....cause this will produces many messages if i use
> > > > > the merging intensive with many branches...
> > > >
> > > > Yeah, this annoyed me a bit as well. Wouldn't it be better to only
> > > > display branches which exist in HEAD by default, and add an option to
> > > > display all branches?
> > >
> > > If i'm right, we are supposed to handle this. In case the entry is not
> > > found, we should display "(source no longer available in HEAD)". I'm not
> > > sure, if the check is supposed to handle this use case anyway.
> >
> > Yeah, they are displayed with "(source no longer available in HEAD)",
> > but it isn't very useful to have tens or hundreds of such entries for
> > feature branches which have been removed ages ago. I'd much rather only
> > have the entries for currently active branches by default.
>
> May I suggest to add "-q" switch for mergeinfo command to address this
> case? If this switch is passed, we can avoid printing the merged and
> eligible revisions for a deleted target.
>
> If this proposal sounds ok, i'll come up with a patch for this.

Sounds good to me. I'd prefer this being the default behaviour, but a
switch is fine.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-03-25 11:35:04 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.