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

Re: svn commit: r11155 - in trunk/subversion: clients/cmdline include libsvn_client libsvn_ra_dav libsvn_ra_local libsvn_ra_svn libsvn_repos mod_dav_svn svnserve tests/clients/cmdline/getopt_tests_data

From: Garrett Rooney <rooneg_at_electricjellyfish.net>
Date: 2004-09-28 14:00:19 CEST

C. Michael Pilato wrote:
> rooneg@tigris.org writes:
>
>
>>Modified: trunk/subversion/clients/cmdline/main.c
>>==============================================================================
>>--- trunk/subversion/clients/cmdline/main.c (original)
>>+++ trunk/subversion/clients/cmdline/main.c Mon Sep 27 20:22:42 2004
>>@@ -144,6 +144,8 @@
>> "with a native svn:eol-style\n"
>> " property. ARG may be one "
>> "of 'LF', 'CR', 'CRLF'")},
>>+ {"limit", 'l', 1,
>>+ N_("maximum number of log entries")},
>> {0, 0, 0, 0}
>> };
>>
>
>
> Whoa, I must have missed a discussion somewhere. I very much dislike
> the idea of eating up a short-form option name ('l') for an option
> that only applies to a single subcommand.

Well, personally I'm in favor of it since when working with large trees
adding a limit on the amount of log messages you're going to see turns
into second nature (I've been running a version of these patches locally
for a little while, and I use it all the time). If enough people object
we can remove the short form though, no huge deal.

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Sep 28 14:00:37 2004

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.