bug report: hg addremove -s -n

Matt Mackall mpm at selenic.com
Sun Jul 22 07:51:26 CDT 2007


On Sun, Jul 22, 2007 at 11:52:06AM +0200, Thomas Arendsen Hein wrote:
> * Dustin Sallings <dustin at spy.net> [20070722 03:37]:
> > 	The error is telling you that it can't parse -n as a float.  It  
> > looks like it wants a number as an argument to -s.
> > 
> > 	If there's a bug, it's in the help output of ``addremove.''
> > 
> > On Jul 21, 2007, at 17:51, Bas van Oostveen wrote:
> > > # hg addremove -s -n
> > > ** unknown exception encountered, details follow
> > > ValueError: invalid literal for float(): -n
> 
> Fixed in crew and crew-stable, it now prints:
> "abort: similarity must be a number"
> 
> The other thing is that the help system should have a way to show
> that some options need a parameter which might have prevented you
> from omitting the number.

This bug might have been better fixed by:

a) teaching fancyopts about floats
b) teaching fancyopts to complain about conversion failures

-- 
Mathematics is the supreme nostalgia of our time.


More information about the Mercurial mailing list