On Wed, Jul 26, 2017 at 02:14:36PM -0000, danie...@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.
