[PATCH 0 of 5 STABLE] largefiles: sync exit codes in overridden commands with documentation

Greg Ward greg at gerg.ca
Thu Sep 6 12:57:22 CDT 2012


On 06 September 2012, Matt Harbison said:
> This started with a fix for 3611, but I found others.  I think I got them all, but another set of eyes wouldn't hurt.  If the function in commands.py didn't explicitly return a value, I didn't bother in largefiles either (e.g. summary).

+0.9 on your patch series as submitted. +1 if you take my suggestion
about the return value of closure warn() in removelargefiles(). ;-)

       Greg
-- 
Greg Ward                                http://www.gerg.ca/
OUR PLAN HAS FAILED STOP JOHN DENVER IS NOT TRULY DEAD
STOP HE LIVES ON IN HIS MUSIC STOP PLEASE ADVISE FULL STOP


More information about the Mercurial-devel mailing list