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

Re: Merge info display

From: Johan Corveleyn <jcorvel_at_gmail.com>
Date: Wed, 5 Oct 2011 10:06:10 +0200

On Wed, Oct 5, 2011 at 5:37 AM, Branko Čibej <brane_at_xbc.nu> wrote:
> On 30.09.2011 16:03, Julian Foad wrote:
>> On Fri, 2011-09-30 at 17:27 +0400, Ivan Zhakov wrote:
>>> On Fri, Sep 30, 2011 at 17:20, Johan Corveleyn <jcorvel_at_gmail.com> wrote:
>>>
>>>> Yes, please keep the old output (one rev per line) available somehow.
>>>> I have developed quite a lot of scripts in our company that use [...]
>>>> If the old style is no longer the default, I can live with that [...]
>>>>
>>> In that case -h (--human-readable) option for svn mergeinfo command
>>> makes sense for me.
>> Of course we can keep the old output available.  Enabling it when the
>> '--show-revs' option is given sounds like a good solution, but we can
>> choose a strictly compatible solution or less strict or whatever, no
>> problem.
>>
>
> I guess ... in this case it just might be better to make the new format
> the default. Assuming the release notes state loudly that scripts that
> use mergeinfo must be updated.
>
> Whatever our policy about backwards compatibility, the primary goal
> should be to make the script useful for humans. Making output
> parsable/useful for scripts is secondary.

Agreed.

Fortunately, all of the script-usage I have is based on "--show-revs
eligible" :-), so if we keep the old output with this option, that
would be great. I have no objections against changing the default
output to something more human-readable.

-- 
Johan
Received on 2011-10-05 10:07:05 CEST

This is an archived mail posted to the Subversion Dev mailing list.