Re: [Bug-wget] Test-504.py sometimes fails on slow machines

2017-02-16 Thread Tim Ruehsen
On Thursday, February 16, 2017 12:39:04 PM CET Adam Sampson wrote: > Hi Tim, > > On Thu, Feb 16, 2017 at 09:33:08AM +0100, Tim Ruehsen wrote: > > Please try this patch, it works out for me even with your additional > > timing > > (thanks for that !) > > It fixes 504 handling in src/url.c and addit

Re: [Bug-wget] Test-504.py sometimes fails on slow machines

2017-02-16 Thread Adam Sampson
Hi Tim, On Thu, Feb 16, 2017 at 09:33:08AM +0100, Tim Ruehsen wrote: > Please try this patch, it works out for me even with your additional timing > (thanks for that !) > It fixes 504 handling in src/url.c and additionally adds Content-Length to > error responses that come with a body. That pat

Re: [Bug-wget] Test-504.py sometimes fails on slow machines

2017-02-16 Thread Tim Ruehsen
On Tuesday, February 14, 2017 4:20:26 PM CET Adam Sampson wrote: > On Tue, Feb 14, 2017 at 04:03:13PM +, Adam Sampson wrote: > > Your patch compiles, but doesn't fix the broken test. tcpdump traces > > attached of the test succeeding and failing with the patch applied (it's > > exactly the same

Re: [Bug-wget] Test-504.py sometimes fails on slow machines

2017-02-14 Thread Tim Rühsen
On Dienstag, 14. Februar 2017 16:20:26 CET Adam Sampson wrote: > On Tue, Feb 14, 2017 at 04:03:13PM +, Adam Sampson wrote: > > Your patch compiles, but doesn't fix the broken test. tcpdump traces > > attached of the test succeeding and failing with the patch applied (it's > > exactly the same b

Re: [Bug-wget] Test-504.py sometimes fails on slow machines

2017-02-14 Thread Adam Sampson
On Tue, Feb 14, 2017 at 04:03:13PM +, Adam Sampson wrote: > Your patch compiles, but doesn't fix the broken test. tcpdump traces > attached of the test succeeding and failing with the patch applied (it's > exactly the same behaviour as before). If it helps: here's a patch to the test server to

Re: [Bug-wget] Test-504.py sometimes fails on slow machines

2017-02-14 Thread Tim Ruehsen
On Tuesday, February 14, 2017 4:03:13 PM CET Adam Sampson wrote: > On Tue, Feb 14, 2017 at 04:31:10PM +0100, Tim Ruehsen wrote: > > I moved the 504 code to where it belongs (IMO) and created the > > attached patch. Please test and report back if it works for you. > > Your patch compiles, but does

Re: [Bug-wget] Test-504.py sometimes fails on slow machines

2017-02-14 Thread Adam Sampson
On Tue, Feb 14, 2017 at 04:31:10PM +0100, Tim Ruehsen wrote: > I moved the 504 code to where it belongs (IMO) and created the > attached patch. Please test and report back if it works for you. Your patch compiles, but doesn't fix the broken test. tcpdump traces attached of the test succeeding and

Re: [Bug-wget] Test-504.py sometimes fails on slow machines

2017-02-14 Thread Tim Ruehsen
On Tuesday, February 14, 2017 2:34:27 PM CET Adam Sampson wrote: > Hi wget maintainers, > > I've just built wget-1.19.1 on several GNU/Linux machines, and found > that Test-504.py fails sometimes on a slow-ish ARMv7 system with Python > 3.6.0. Attached are tcpdump traces of the test succeeding and