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
> ]]]
Received on 2013-07-01 23:52:48 CEST