On Tue, Jul 02, 2013 at 12:37:44AM +0300, Daniel Shahaf wrote: > 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.
There wasn't just --sca. There was a huge list of aliases that got removed again. See http://svn.apache.org/r1126808 log message and discussion linked from there. I think that adding new entries to the short options list re-opens that discussion. I don't really want to re-open it myself. If we re-open it I think we should come to an agreement about whether we still want to add short alises, and if so what kinds of aliases we want. And then put a comment into svn.c that spot to document that decision. > #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}, > + {"rm-?", opt_remove_unversioned, 0, NULL}, > > {0, 0, 0, 0}, > }; > ]]] > > [[[ > % touch notes/foo.o > % $svn cleanup --rm-I notes > D notes/foo.o > ]]]
