Re: Revision number syntax in merge notifications.

Mattias Engdegård wrote:
> "C. Michael Pilato" <cmpilato@collab.net> writes:
>
>> Merging r3
>> Merging r5:16
>
>> instead of:
>
>> Merging r3
>> Merging r5-r16
>
> Is c5 (r4:5) included in the merge? If so, r5:16 is incorrect or at least
> inconsistent.
> svnmerge.py includes the lower bound in its -r5-16 notation; perhaps it should
> use -c5-16 instead.

You have a good point there, followed up by a bad suggestion. :-)

-r5:16 does *not* include revision 5. I don't know if that is consistent
with Ben's reportion intentions or not. Ben?

But -c5-16 means nothing, and I'd rather read "revisions 5-16" (which has no
command-line-option utility, but has a crystal-clear meaning) than something
which looks like a command-line option but ain't.

-- 
C. Michael Pilato <cmpilato@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand