[PATCH 4 of 4 stable] largefiles: don't mute/obfuscate http errors when putlfile fails
Mads Kiilerich
mads at kiilerich.com
Thu Oct 22 06:12:33 CDT 2015
On 10/22/2015 12:35 PM, Augie Fackler wrote:
> On Wed, Oct 21, 2015 at 09:16:07PM +0200, Mads Kiilerich wrote:
>> # HG changeset patch
>> # User Mads Kiilerich <madski at unity3d.com>
>> # Date 1445454338 -7200
>> # Wed Oct 21 21:05:38 2015 +0200
>> # Branch stable
>> # Node ID 672fe15eacfa6cb600febf2e0ad48e9f9fd81c18
>> # Parent 45a74ff314b78e2539049bbf3e90d59e9a6c90dd
>> largefiles: don't mute/obfuscate http errors when putlfile fails
> Looks good, but these seem a bit new-feature-y and not
> regression-fix-y for stable to me. mpm?
I agree they not are regression fixes and thus could be postponed; it
fixes stuff that was introduced by this 'various' dude who introduced
largefiles.
I do however consider it a bug that store corruption could go unnoticed
and be propagated to the working directory where it caused an
"impossible" "--clean leave working dir with [incorrect] stuff to
commit". The fix for that is appropriate for stable.
Incorrect and misleading reporting of bugs also seems like a bug to me.
/Mads
More information about the Mercurial-devel
mailing list