[PATCH 00 of 19 STABLE] largefiles: fix bugs of "hg status"
Greg Ward
greg at gerg.ca
Tue Feb 28 01:38:54 UTC 2012
On 27 February 2012, FUJIWARA Katsunori said:
> this patch series fixes bugs of "hg status" with largefiles extension.
>
> many of patches in this series are refactoring ones as preparation for
> bugfix in #7, #10, #12, #14, #16, #18, and #19.
I've looked over the first couple of patches, up to #7. The big
question: *which* bugs are you fixing? Are these bugs already in
Mercurial's issue tracker? If so, you should put " (issueXXXX)" in the
commit message. If not, the commit message should clearly explain what
the bug is. From patch #7, I don't understand what's broken in the
current code or how you fixed it. A clearer commit message would help.
A test case would help even more.
Also, refactoring is generally frowned upon in the stable branch. If
the bugs in question cannot be fixed without refactoring, so be it:
the fixes will just have to wait for Mercurial 2.2. But if it's
possible to fix on stable and refactor on default, that increases the
chance of your fixes getting to users sooner (e.g. 2.1.1).
Greg
--
Greg Ward http://www.gerg.ca/
Very few profundities can be expressed in less than 80 characters.
More information about the Mercurial-devel
mailing list