Re: RFC: svn mergeinfo improvements for 1.5

On Tue, Apr 1, 2008 at 11:03 AM, Mark Phippard <markphip_at_gmail.com> wrote:
> On Tue, Apr 1, 2008 at 10:07 AM, Julian Foad <julianfoad_at_btopenworld.com> wrote:
> > Hi, "svn mergeinfo" folks.
> >
> > I can see two alternative approaches:
> >
> > (1) Make an initial implementation of "svn mergeinfo" that tries to be a bare
> > minimum data provider.
>
> While some of your other ideas seem pretty good, I think at this point
> we are probably best off sticking with this sort of idea. Let's just
> take the simple approach. That is basically how the command currently
> works, and this is really just about making the output more accessible
> rather than extending or changing the output at this point.
>
> The key is that the existing API is solid and usable for GUI's. We
> should probably keep the CLI simple until we are sure about what we
> want to implement.

+1 to going to simple route and just displaying the bare minimum and
then working on a more elaborate CLI in 1.6.

Here is my (probably futile) attempt to define what the bare minimum is:

A) Revisions are displayed one per line (no ranges)

B) Inoperative revisions are filtered out, but for merged and eligible
revisions*

C) Support only one TARGET path/URL

D) Require --from-source.

E) Show merged revisions by default. Add a new option --show-revs
['merged' | 'eligible'] allowing eligible to be shown instead.

F) Make recursive by default, i.e. any subtrees of the TARGET with
their own explicit mergeinfo are listed separately. This implies at
least one level of indentation.

*I believe Mike has already done some of the heavy lifting for the
filtering on the svn-mergeinfo-enhancements branch, so we are probably
not that far off.

Paul

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-01 17:27:06 CEST