You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Justin Erenkrantz <je...@ebuilt.com> on 2002/01/19 17:40:03 UTC

Re: svn commit: rev 966 - trunk/subversion/clients/cmdline trunk/subversion/tests/clients/cmdline

On Sat, Jan 19, 2002 at 11:14:16AM -0600, sussman@tigris.org wrote:
> 3.  Create short form of --force (-f), which didn't exist.  (?!)

I don't know, but I wonder what force would do on a commit?  =)

I'm just envisioning someone accidentally typing:

svn commit -f /path/to/log

instead of

svn commit -F /path/to/log

To me, this seems like an inevitable typo.  Perhaps, -f is not an
option for commit?  

(My brain isn't awake enough to parse SVN's code to verify this
myself.)  -- justin


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn commit: rev 966 - trunk/subversion/clients/cmdline trunk/subversion/tests/clients/cmdline

Posted by Ben Collins-Sussman <su...@collab.net>.
Justin Erenkrantz <je...@ebuilt.com> writes:

> To me, this seems like an inevitable typo.  Perhaps, -f is not an
> option for commit?  
> 
> (My brain isn't awake enough to parse SVN's code to verify this
> myself.)  -- justin

No need to parse the code:  

[sussman@kepler:~/projects/svn]$ svn help ci
commit (ci): Send changes from your working copy to the repository.
usage: svn commit [TARGETS]

   Be sure to use one of -m or -F to send a log message;
   the -r switch is only for use with --xml-file.

Valid options:
  -m [--message] arg:   specify commit message "ARG"
  -F [--file] arg:   read data from file ARG
  -q [--quiet]:   print as little as possible
  -f [--force]:   force operation to run
  --username arg:   specify a username ARG
  --password arg:   specify a password ARG
  --xml-file arg:   read/write xml to specified file ARG
  -r [--revision] arg:   specify revision number ARG (or X:Y range)


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn commit: rev 966 - trunk/subversion/clients/cmdline trunk/subversion/tests/clients/cmdline

Posted by Garrett Rooney <ro...@electricjellyfish.net>.
On Sat, Jan 19, 2002 at 02:10:23PM -0800, Greg Stein wrote:
> I'm with Justin.
> 
> -1 on allowing -f for the force command. If you want to *force* something,
> then you darned well better spell that out. The short form just makes it too
> easy for an accident.

i agree.  if the user wants to shoot themselves in the foot, that's fine, 
but lets make them load the gun themselves.

-garrett

-- 
garrett rooney                     Unix was not designed to stop you from 
rooneg@electricjellyfish.net       doing stupid things, because that would  
http://electricjellyfish.net/      stop you from doing clever things.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn commit: rev 966 - trunk/subversion/clients/cmdline trunk/subversion/tests/clients/cmdline

Posted by Greg Stein <gs...@lyra.org>.
I'm with Justin.

-1 on allowing -f for the force command. If you want to *force* something,
then you darned well better spell that out. The short form just makes it too
easy for an accident.

Cheers,
-g

On Sat, Jan 19, 2002 at 09:40:03AM -0800, Justin Erenkrantz wrote:
> On Sat, Jan 19, 2002 at 11:14:16AM -0600, sussman@tigris.org wrote:
> > 3.  Create short form of --force (-f), which didn't exist.  (?!)
> 
> I don't know, but I wonder what force would do on a commit?  =)
> 
> I'm just envisioning someone accidentally typing:
> 
> svn commit -f /path/to/log
> 
> instead of
> 
> svn commit -F /path/to/log
> 
> To me, this seems like an inevitable typo.  Perhaps, -f is not an
> option for commit?  
> 
> (My brain isn't awake enough to parse SVN's code to verify this
> myself.)  -- justin
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org

-- 
Greg Stein, http://www.lyra.org/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org