Re: svn commit: r1803053 - /subversion/trunk/subversion/svn/svn.c

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: svn commit: r1803053 - /subversion/trunk/subversion/svn/svn.c

Stefan Sperling
On Wed, Jul 26, 2017 at 02:14:36PM -0000, [hidden email] wrote:

> Author: danielsh
> Date: Wed Jul 26 14:14:36 2017
> New Revision: 1803053
>
> URL: http://svn.apache.org/viewvc?rev=1803053&view=rev
> Log:
> * subversion/svn/svn.c
>   (svn_cl__cmd_table."cleanup"): Edit the help text so it is clear that
>     breaking locks does not apply to the newer modes of operation.
>
> Modified:
>     subversion/trunk/subversion/svn/svn.c
>
> Modified: subversion/trunk/subversion/svn/svn.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/svn/svn.c?rev=1803053&r1=1803052&r2=1803053&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/svn/svn.c (original)
> +++ subversion/trunk/subversion/svn/svn.c Wed Jul 26 14:14:36 2017
> @@ -609,35 +609,32 @@ const svn_opt_subcommand_desc2_t svn_cl_
>      {'r', 'q', 'N', opt_depth, opt_force, opt_ignore_externals} },
>  
>    { "cleanup", svn_cl__cleanup, {0}, N_
> -    ("Recursively clean up the working copy, removing write locks, resuming\n"
> -     "unfinished operations, removing unversioned or ignored items, etc.\n"
> +    ("Either recover from an interrupted operation that left the working copy locked,\n"
> +     "or remove unwanted files.\n"
>       "usage: 1. cleanup [WCPATH...]\n"
> -     "usage: 2. cleanup OPTIONS [WCPATH...]\n"
> +     "       2. cleanup --remove-unversioned [WCPATH...]\n"
> +     "          cleanup --remove-ignored [WCPATH...]\n"
> +     "       3. cleanup --vacuum-pristines [WCPATH...]\n"

Thanks, that is indeed clearer.

Technically, both --remove-* options and --vacuum-pristines can be combined.
But I don't think we need to be that precise in the help text, so I don't
see a reason to adjust your change.
Loading...