Re: [PATCH] add 'svn version' as 'svnversion' v1

Dear Max,

>> Create a 'version' subcommand so that 'svn version' behaves as
>> 'svnversion' does.
>>
>> QUESTIONS
>>
>> - could the functionnality be moved to some library and invoked there
>> by both 'svn' and 'svnversion'? If so, which one?
>
> libsvn_client

Ok. Make sense.

>> - would the subcommand be better named 'revision/revisions' or some other
>> name? Here there is a subcommand and an option with the same name...
>
> Yes, if we are going to do this, we should *definitely* take the opportunity
> to choose a better name.

Sure.

Well, "revisions" looks ok to me, but I'm not a native speaker.
Moreover, there is already a --revision option.

I guess it is possible to provide a better solution, but to let
"svn version" anyway as an alias as people are used to it.

What about: rev[ision]-?stat[us], vers?[ion]-?stat[us]...

>> - what about 'svn rm subversion/svnversion'?
>> add svnversion = exec svn version as a substitute?
>
> No, just push the functionality down into libsvn_client, and let both svn and
> svnversion call it.

Ok. I will submit again something along these lines...

Thanks,

-- 
Fabien.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org