On Wed, Aug 27, 2008 at 03:09:15PM -0400, Karl Fogel wrote:
> Neels Hofmeyr <neels_at_elego.de> writes:
> > The second one could be paraphrased: "To make it easier to type a
> > subcommand". IMHO, `setprop' sits much better in my brain, to the degree
> > that it involuntarily turns `propset' around do `setprop'.
> >
> > Once that's there, I'd expect `sp' to work as well.
>
> But alias space doesn't come free, remember. Someday later, we might
> want 'sp' for "show patch". Or whatever, I don't know -- the point is,
> if we expand too many commands into large equivalence classes, those
> classes will inevitably start to interfere with future commands.
>
> I think that introducing "setprop" is not actually a good idea. The
> vast majority of people are not even using the command-line client at
> all; of those who are, very few have complained about this. I
> understand that for them it's a real issue -- I am not trying to deny
> this. But I'm not sure the cost of fixing it is worth the gain. Having
> "setprop" does imply having "sp"; having "sp"; likewise with "getprop"
> and "gp". We shouldn't close off alias space for such small gains.
Another obvious alternative to a patch to Subversion would be
an 'svn' wrapper script that does the setprop -> propset etc.
translation.
Given that apparently only few people really care about this, and
that there is resistence in the developer community towards this change,
should we just tell people to create such a wrapper script / shell
function instead?
If so, we could close issue #3268 as WONTFIX.
I personally still think it's a usability issue with our command
line client, and would like to see the whole set of aliases added.
But there is no point in adding them unless there is consensus among
developers about this change, which does not seem to be in sight.
Stefan
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-08-28 13:03:32 CEST