[PATCH 00 of 19 STABLE] largefiles: fix bugs of "hg status"
FUJIWARA Katsunori
foozy at lares.dti.ne.jp
Tue Feb 28 07:32:58 UTC 2012
At Mon, 27 Feb 2012 20:38:54 -0500,
Greg Ward wrote:
>
> 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).
Thank you for reading patches and your comments, Greg.
I think refactorings are still needed to simplify bug fix patches.
So, I'll try to re-fine patches themselves, description of them and
test code, and post them for 'default'.
----------------------------------------------------------------------
[FUJIWARA Katsunori] foozy at lares.dti.ne.jp
More information about the Mercurial-devel
mailing list