On Wed, Jul 26, 2017 at 02:14:36PM -0000, danielsh_at_apache.org 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.
Received on 2017-07-26 16:17:58 CEST