Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-28 Thread Evangelos Foutras
The updated Test-proxied-https-auth.px fails very often for me. Log from the failing test is attached; using Perl 5.20.1 on Arch Linux. converted 'https://no.such.domain/needs-auth.txt' (ANSI_X3.4-1968) -> 'https://no.such.domain/needs-auth.txt' (UTF-8) --2014-10-29 02:21:13-- https://no.such.dom

Re: [Bug-wget] [PATCH] Add valgrind testing support via ./configure

2014-10-28 Thread Pär Karlsson
Regarding the Perl test suite, I'd be happy to clean up the code a little bit, if you think it's worth it? There are some minor issues with the current code (such as perl producing certain warnings regarding invalid operators, i.e: "Argument "" isn't numeric in numeric eq (==) at WgetTests.pm line

Re: [Bug-wget] broken progressbar in 1.16

2014-10-28 Thread Michael Shigorin
On Tue, Oct 28, 2014 at 09:22:14PM +0100, Giuseppe Scrivano wrote: > don't you think you are a bit exaggerating for the minor issue > you have reported? I see a pattern unfortunately being ALT Linux wget maintainer since 2005 (and using it since last century). GNU projects like this used to do no

Re: [Bug-wget] broken progressbar in 1.16

2014-10-28 Thread Giuseppe Scrivano
Michael Shigorin writes: > Hello, > please do pay attention to regression testing for innovative > bells and especially whistles as the "new and improved" progress > bar quickly becomes annoying if the filename is longer than the > part of the line before percent meter: the new code tries t

Re: [Bug-wget] [PATCH] Fixing several memory leaks

2014-10-28 Thread Tim Ruehsen
On Tuesday 28 October 2014 13:26:44 Giuseppe Scrivano wrote: > Tim Ruehsen writes: > > On Thursday 23 October 2014 22:04:55 Tim Rühsen wrote: > >> Am Donnerstag, 9. Oktober 2014, 12:28:05 schrieb Tim Ruehsen: > >> > Hi, > >> > > >> > using the new valgrind testing feature (yet not completely disc

Re: [Bug-wget] broken progressbar in 1.16

2014-10-28 Thread Yousong Zhou
Hi On 28 October 2014 19:38, Michael Shigorin wrote: > On Tue, Oct 28, 2014 at 03:16:00PM +0800, Darshit Shah wrote: >> While we try our best to regression test every new feature, >> some issues often seep through the cracks. > > I do understand that, hence RC proposition (would be great to > ha

Re: [Bug-wget] [PATCH] Fixing several memory leaks

2014-10-28 Thread Giuseppe Scrivano
Tim Ruehsen writes: > On Thursday 23 October 2014 22:04:55 Tim Rühsen wrote: >> Am Donnerstag, 9. Oktober 2014, 12:28:05 schrieb Tim Ruehsen: >> > Hi, >> > >> > using the new valgrind testing feature (yet not completely discussed), I >> > found and fixed several memory leaks. >> > >> > Also, I

Re: [Bug-wget] [PATCH] Fixing several memory leaks

2014-10-28 Thread Tim Ruehsen
On Thursday 23 October 2014 22:04:55 Tim Rühsen wrote: > Am Donnerstag, 9. Oktober 2014, 12:28:05 schrieb Tim Ruehsen: > > Hi, > > > > using the new valgrind testing feature (yet not completely discussed), I > > found and fixed several memory leaks. > > > > Also, I found that parsing the proxy UR

Re: [Bug-wget] broken progressbar in 1.16

2014-10-28 Thread Michael Shigorin
On Tue, Oct 28, 2014 at 03:16:00PM +0800, Darshit Shah wrote: > While we try our best to regression test every new feature, > some issues often seep through the cracks. I do understand that, hence RC proposition (would be great to have distro maintainers and other interested grumblers join some a

Re: [Bug-wget] [PATCH] Add valgrind testing support via ./configure

2014-10-28 Thread Tim Ruehsen
On Monday 27 October 2014 21:59:28 Tim Rühsen wrote: > Am Samstag, 25. Oktober 2014, 23:40:41 schrieb Tim Rühsen: > > Am Sonntag, 19. Oktober 2014, 17:41:30 schrieb Darshit Shah: > > > On 10/09, Tim Rühsen wrote: > > > >> Hence, hard coding the command actually reduces the amount of work a > > > >>

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-28 Thread Tim Ruehsen
On Monday 27 October 2014 12:50:00 Tim Ruehsen wrote: > On Monday 27 October 2014 12:09:45 Tim Ruehsen wrote: > > On Monday 27 October 2014 11:51:24 Tim Ruehsen wrote: > > > On Sunday 26 October 2014 21:29:14 Darshit Shah wrote: > > > > Ever since the Perl tests have been ported to the parallel tes

Re: [Bug-wget] COMMENT_IF_NO_POD2MAN broken?

2014-10-28 Thread Giuseppe Scrivano
Tim Ruehsen writes: > On Monday 27 October 2014 11:23:45 Dan McDonald wrote: >> Hello from OmniOS (an illumos distro). Giuseppe told me to send this report >> here. >> >> Consider this diff: >> >> >> http://git.savannah.gnu.org/cgit/wget.git/commit/doc/Makefile.am?id=d1ab00c >> ab4d644d1048

Re: [Bug-wget] COMMENT_IF_NO_POD2MAN broken?

2014-10-28 Thread Tim Ruehsen
On Monday 27 October 2014 11:23:45 Dan McDonald wrote: > Hello from OmniOS (an illumos distro). Giuseppe told me to send this report > here. > > Consider this diff: > > > http://git.savannah.gnu.org/cgit/wget.git/commit/doc/Makefile.am?id=d1ab00c > ab4d644d1048ed279f90e633b82cf4455 Thanks fo

Re: [Bug-wget] broken progressbar in 1.16

2014-10-28 Thread Darshit Shah
On 10/28, Darshit Shah wrote: Hi Michael, Thanks a lot for your email. While we try our best to regression test every new feature, some issues often seep through the cracks. I'm traveling so testing this is a little hard for me. But I managed to do some preliminary testing and identified th

Re: [Bug-wget] broken progressbar in 1.16

2014-10-28 Thread Darshit Shah
Hi Michael, Thanks a lot for your email. While we try our best to regression test every new feature, some issues often seep through the cracks. On 28-Oct-2014 2:13 pm, "Michael Shigorin" wrote: > > Hello, > please do pay attention to regression testing for innovative > bells and especia