Konstantin Kolinko wrote on Wed, Jul 03, 2013 at 02:22:54 +0400:
> 2013/7/2 Daniel Shahaf <d.s_at_daniel.shahaf.name>:
> > How about adding 'svn cleanup --rm-I' as a short option for 'svn cleanup
> > --remove-ignored'?
> >
> > Currently we have --cl as a short option for --changelist, and in the
> > past we decided not to add --sca as a short option for --show-copies-as-adds.
> >
> > #svn-dev asked me to post here before committing, so here.
> >
> >
> > [[[
> > Index: subversion/svn/svn.c
> > ===================================================================
> > --- subversion/svn/svn.c (revision 1498689)
> > +++ subversion/svn/svn.c (working copy)
> > @@ -393,6 +393,8 @@ const apr_getopt_option_t svn_cl__options[] =
> > */
> >
> > {"cl", opt_changelist, 1, NULL},
> > + {"rm-I", opt_remove_ignored, 0, NULL},
>
> Is it actually an uppercase 'i' from 'ignored'? It looks a lot like
> lowercase 'L'.
Yes, those are ASCII 63 and 0x49, from the first column of 'svn status'.
That is also the reason for mixing upper/lower case in the same argument:
printf("--rm-%c", svn_status_lines[N][0]);.
(and yes, I knew the question mark would be a hard sell :-))
Received on 2013-07-03 13:45:56 CEST