Re: --rm-I alias to --remove-ignored

2013-07-03 Thread Daniel Shahaf
Konstantin Kolinko wrote on Wed, Jul 03, 2013 at 02:22:54 +0400: > 2013/7/2 Daniel Shahaf : > > 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 ad

Re: --rm-I alias to --remove-ignored

2013-07-02 Thread Justin Erenkrantz
On Mon, Jul 1, 2013 at 5:37 PM, Daniel Shahaf wrote: > How about adding 'svn cleanup --rm-I' as a short option for 'svn cleanup > --remove-ignored'? > I don't like either option (rm-I or rm-?) as I don't find either an improvement over the long option. Mixing lower-and-upper in the same argument

Re: --rm-I alias to --remove-ignored

2013-07-02 Thread Konstantin Kolinko
2013/7/2 Daniel Shahaf : > 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

Re: --rm-I alias to --remove-ignored

2013-07-01 Thread C. Michael Pilato
On 07/01/2013 05:37 PM, Daniel Shahaf wrote: > How about adding 'svn cleanup --rm-I' as a short option for 'svn cleanup > --remove-ignored'? File an ENHANCEMENT issue with a note to evaluate later whether there is sufficient user demand for the option? -- C. Michael Pilato CollabNet <> www

Re: --rm-I alias to --remove-ignored

2013-07-01 Thread Daniel Shahaf
Stefan Sperling wrote on Tue, Jul 02, 2013 at 00:29:14 +0200: > On Tue, Jul 02, 2013 at 01:14:45AM +0300, Daniel Shahaf wrote: > > Daniel Shahaf wrote on Tue, Jul 02, 2013 at 01:12:17 +0300: > > > Stefan Sperling wrote on Mon, Jul 01, 2013 at 23:52:10 +0200: > > > > On Tue, Jul 02, 2013 at 12:37:44

Re: --rm-I alias to --remove-ignored

2013-07-01 Thread Stefan Sperling
On Tue, Jul 02, 2013 at 01:14:45AM +0300, Daniel Shahaf wrote: > Daniel Shahaf wrote on Tue, Jul 02, 2013 at 01:12:17 +0300: > > Stefan Sperling wrote on Mon, Jul 01, 2013 at 23:52:10 +0200: > > > On Tue, Jul 02, 2013 at 12:37:44AM +0300, Daniel Shahaf wrote: > > > > How about adding 'svn cleanup -

Re: --rm-I alias to --remove-ignored

2013-07-01 Thread Daniel Shahaf
Daniel Shahaf wrote on Tue, Jul 02, 2013 at 01:12:17 +0300: > Stefan Sperling wrote on Mon, Jul 01, 2013 at 23:52:10 +0200: > > 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'? > >

Re: --rm-I alias to --remove-ignored

2013-07-01 Thread Daniel Shahaf
Stefan Sperling wrote on Mon, Jul 01, 2013 at 23:52:10 +0200: > 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

Re: --rm-I alias to --remove-ignored

2013-07-01 Thread Stefan Sperling
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-c

Re: --rm-I alias to --remove-ignored

2013-07-01 Thread Ivan Zhakov
On Tue, Jul 2, 2013 at 1:37 AM, Daniel Shahaf wrote: > How about adding 'svn cleanup --rm-I' as a short option for 'svn cleanup > --remove-ignored'? > It's better to leave only long option name for '--remove-ignored' since it's potentially dangerous/destructive command. -- Ivan Zhakov CTO | Visu

--rm-I alias to --remove-ignored

2013-07-01 Thread Daniel Shahaf
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