Re: svn commit: r1228371 - in /subversion/trunk/subversion: svnadmin/main.c tests/cmdline/svnadmin_tests.py tests/cmdline/svntest/main.py

On Fri, Jan 6, 2012 at 14:50, <cmpilato_at_apache.org> wrote:
> Author: cmpilato
> Date: Fri Jan  6 19:50:30 2012
> New Revision: 1228371
>
> URL: http://svn.apache.org/viewvc?rev=1228371&view=rev
> Log:
> Finish issue #4092 ("Add 'svnadmin unlock' subcommand which provides a
> lock token for unlock validation").

The requirement to specify a username/token seems a bit silly. If you
have access to svnadmin, then you can just 'svnadmin lslocks' and feed
that information into the command.

I would recommend simplifying the command to just require the path.
This is the *administrator* that you're talking about. The
double-checks for a username/token are a bit silly for the admin.

Cheers,
-g
Received on 2012-01-06 21:57:01 CET