[PATCH stable] mq: fix qrefresh case sensitivity (issue3271)

Matt Mackall mpm at selenic.com
Fri Nov 2 10:29:12 CDT 2012


On Fri, 2012-11-02 at 15:25 +0100, Adrian Buehlmann wrote:

> >    $ hg qrefresh nonexistentfilename # clear patch
> > +  nonexistentfilename: No such file or directory

> The preexisting comment ("# clear patch") is probably a hint that
> someone tried to "clear the patch".
> 
> But it seems to me that this didn't clear the patch.

It did. It refreshed the patch including only nonexistent files. It thus
contains no changes.

> It seems it just silently failed before this patch, now we get an error
> message.

 $ hg qref nosuch; echo $?
 nosuch: No such file or directory
 0

..continues to clear the patch, but now issuing a pretty reasonable
warning.

A more sensible way to do this might be:

 hg qref -X .

which might be a good thing to add in the help examples for qref. But I
think we should leave the test command as is (with a glob).

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list