<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 24, 2017 at 12:10 PM, Augie Fackler <span dir="ltr"><<a href="mailto:raf@durin42.com" target="_blank">raf@durin42.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On Apr 24, 2017, at 15:05, Gregory Szorc <<a href="mailto:gregory.szorc@gmail.com">gregory.szorc@gmail.com</a>> wrote:<br>
><br>
> On Mon, Apr 24, 2017 at 11:39 AM, Augie Fackler <<a href="mailto:raf@durin42.com">raf@durin42.com</a>> wrote:<br>
> # HG changeset patch<br>
> # User Augie Fackler <<a href="mailto:augie@google.com">augie@google.com</a>><br>
> # Date 1493059110 14400<br>
> #      Mon Apr 24 14:38:30 2017 -0400<br>
> # Branch stable<br>
> # Node ID 52cca17ac523b490acdab259e794e7<wbr>c2898efeda<br>
> # Parent  40cf693fc07d846502f9c15a160288<wbr>0ca99d7b56<br>
> tests: add lots of regex matching to test-http-bad-server.t<br>
><br>
> This makes the test pass in Python 2.6. It's all one of two problems:<br>
><br>
> 1)<br>
>   abort: error: '' (2.7)<br>
>   vs<br>
>   abort: error: (2.6)<br>
><br>
> 2) 65537 (2.7) vs -1 (2.6)<br>
><br>
> With some...unfortunate escaping required due to how regular<br>
> expressions work. Sigh.<br>
><br>
> Ugh. I'd be tempted to require 2.7 to run the test.<br>
<br>
</span>I thought about that. For a counterproposal, how about we decide that 4.2 (or 4.3, if we're feeling generous) is the last version to support 2.6, and we plan to back this "fix" out when we drop 2.6?<br></blockquote><div><br></div><div>Please start a new thread with that proposal! FWIW, I support making 4.2 the last release with 2.6 support.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
><br>
> Another alternative is to monkeypatch readline() so it does the right thing. We already have hacks in keepalive.py to work around bugs in the stdlib.<br>
<br>
</span>eww<br>
<span class=""><br>
> FWIW since this test is already on the flaky side, I was considering just disabling it wholesale on stable until we can make it more robust next cycle.<br>
<br>
</span>Eh, I think the test has merit. I'd rather just tweak it so it passes and do something more serious in the 4.3 cycle.<br></blockquote><div><br></div><div>OK. Then this patch - ugly as it is - is acceptable to me. I can't wait to revert it once 2.6 support is dropped.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>  diff --git a/tests/test-http-bad-server.t b/tests/test-http-bad-server.t<br>
[...]<br>
<br>
</blockquote></div><br></div></div>