You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by "C. Michael Pilato" <cm...@collab.net> on 2006/04/04 20:28:56 UTC

Re: svn commit: r19178 - in trunk/subversion/tests/cmdline: . svntest

dlr@tigris.org wrote:
> Author: dlr
> Date: Tue Apr  4 13:20:34 2006
> New Revision: 19178
> 
> Modified:
>    trunk/subversion/tests/cmdline/README
>    trunk/subversion/tests/cmdline/svntest/main.py
> 
> Log:
> Add a "--list" option (with the same meaning as the "list"
> subcommand), which deprecates but does not remove the "list"
> subcommand.

In my opinion, there is benefit to keeping the Python and C test frameworks
as much in sync as possible (where doing so doesn't cripple one or the other).

Bitesize task for someone -- teach .../tests/svn_test_main.c to:

   * use APR's command-line argument parsing code
   * complain about bad input (such as "--list", today)
   * accept "--fstype [bdb|fsfs]" instead of only --fstype=[bdb|fsfs]"

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

Re: svn commit: r19178 - in trunk/subversion/tests/cmdline: . svntest

Posted by Kamesh Jayachandran <ka...@collab.net>.
Filed a issue 2532.

Will post a patch soon.

With regards
Kamesh Jayachandran
Garrett Rooney wrote:
> On 4/4/06, C. Michael Pilato <cm...@collab.net> wrote:
>   
>> dlr@tigris.org wrote:
>>     
>>> Author: dlr
>>> Date: Tue Apr  4 13:20:34 2006
>>> New Revision: 19178
>>>
>>> Modified:
>>>    trunk/subversion/tests/cmdline/README
>>>    trunk/subversion/tests/cmdline/svntest/main.py
>>>
>>> Log:
>>> Add a "--list" option (with the same meaning as the "list"
>>> subcommand), which deprecates but does not remove the "list"
>>> subcommand.
>>>       
>> In my opinion, there is benefit to keeping the Python and C test frameworks
>> as much in sync as possible (where doing so doesn't cripple one or the other).
>>
>> Bitesize task for someone -- teach .../tests/svn_test_main.c to:
>>
>>    * use APR's command-line argument parsing code
>>    * complain about bad input (such as "--list", today)
>>    * accept "--fstype [bdb|fsfs]" instead of only --fstype=[bdb|fsfs]"
>>     
>
> +1!  The --fs-type=bdb vs --fs-type bdb thing annoys me constantly!
>
> -garrett
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
>
>   


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

Re: svn commit: r19178 - in trunk/subversion/tests/cmdline: . svntest

Posted by Daniel Rall <dl...@collab.net>.
On Tue, 04 Apr 2006, Garrett Rooney wrote:

> On 4/4/06, C. Michael Pilato <cm...@collab.net> wrote:
> > dlr@tigris.org wrote:
> > > Author: dlr
> > > Date: Tue Apr  4 13:20:34 2006
> > > New Revision: 19178
> > >
> > > Modified:
> > >    trunk/subversion/tests/cmdline/README
> > >    trunk/subversion/tests/cmdline/svntest/main.py
> > >
> > > Log:
> > > Add a "--list" option (with the same meaning as the "list"
> > > subcommand), which deprecates but does not remove the "list"
> > > subcommand.
> >
> > In my opinion, there is benefit to keeping the Python and C test frameworks
> > as much in sync as possible (where doing so doesn't cripple one or the other).
> >
> > Bitesize task for someone -- teach .../tests/svn_test_main.c to:
> >
> >    * use APR's command-line argument parsing code
> >    * complain about bad input (such as "--list", today)
> >    * accept "--fstype [bdb|fsfs]" instead of only --fstype=[bdb|fsfs]"
> 
> +1!  The --fs-type=bdb vs --fs-type bdb thing annoys me constantly!

+1 (and you might even have hooked me).
-- 

Daniel Rall

Re: svn commit: r19178 - in trunk/subversion/tests/cmdline: . svntest

Posted by Garrett Rooney <ro...@electricjellyfish.net>.
On 4/4/06, C. Michael Pilato <cm...@collab.net> wrote:
> dlr@tigris.org wrote:
> > Author: dlr
> > Date: Tue Apr  4 13:20:34 2006
> > New Revision: 19178
> >
> > Modified:
> >    trunk/subversion/tests/cmdline/README
> >    trunk/subversion/tests/cmdline/svntest/main.py
> >
> > Log:
> > Add a "--list" option (with the same meaning as the "list"
> > subcommand), which deprecates but does not remove the "list"
> > subcommand.
>
> In my opinion, there is benefit to keeping the Python and C test frameworks
> as much in sync as possible (where doing so doesn't cripple one or the other).
>
> Bitesize task for someone -- teach .../tests/svn_test_main.c to:
>
>    * use APR's command-line argument parsing code
>    * complain about bad input (such as "--list", today)
>    * accept "--fstype [bdb|fsfs]" instead of only --fstype=[bdb|fsfs]"

+1!  The --fs-type=bdb vs --fs-type bdb thing annoys me constantly!

-garrett

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