[Bug-wget] Fix "end" range check in map_html_tags()

2014-11-01 Thread Witchakorn Kamolpornwijit
-- *Witchakorn Kamolpornwijit* MIT Class of 2016 Blog: http://chalet16.com Portfolio: http://www6.portfolio.chalet16.com Facebook/Twitter/IRC: chalet16 Skype: chalet1601 0001-Fix-range-check-in-map_html_tags.patch Description: Binary data

Re: [Bug-wget] broken progressbar in 1.16

2014-11-01 Thread Michael Shigorin
grown testing one) http://fly.osdn.org.ua/~mike/iso/tmp/live-flightgear-20141101-x86_64.iso Second run resulted in 1002020 bytes downloaded with the same failure. --   WBR, Michael Shigorin / http://altlinux.org   -- http://opennet.ru / http://anna-news.info

Re: [Bug-wget] broken progressbar in 1.16

2014-11-01 Thread Darshit Shah
On 11/01, Giuseppe Scrivano wrote: Darshit Shah writes: From 1a29f21bb24c531f450dd59446a0c5a0f7e81110 Mon Sep 17 00:00:00 2001 From: Darshit Shah Date: Sat, 1 Nov 2014 05:34:04 +0530 Subject: [PATCH] Remove extra padding from the progress bar --- src/ChangeLog | 7 +++ src/progress.c |

Re: [Bug-wget] [PATCH 1/5] use pkg-config to check zlib deps if available

2014-11-01 Thread Darshit Shah
On 11/01, Giuseppe Scrivano wrote: Hi Darshit, Darshit Shah writes: On 11/01, Giuseppe Scrivano wrote: Darshit Shah writes: On 10/31, Mike Frysinger wrote: On 31 Oct 2014 16:50, Giuseppe Scrivano wrote: Mike Frysinger writes: > Newer versions of these packages ship with pkg-config file

Re: [Bug-wget] [PATCH 1/5] use pkg-config to check zlib deps if available

2014-11-01 Thread Giuseppe Scrivano
Hi Darshit, Darshit Shah writes: > On 11/01, Giuseppe Scrivano wrote: >>Darshit Shah writes: >> >>> On 10/31, Mike Frysinger wrote: On 31 Oct 2014 16:50, Giuseppe Scrivano wrote: > Mike Frysinger writes: > > Newer versions of these packages ship with pkg-config files, so if we >>>

Re: [Bug-wget] broken progressbar in 1.16

2014-11-01 Thread Giuseppe Scrivano
Darshit Shah writes: > From 1a29f21bb24c531f450dd59446a0c5a0f7e81110 Mon Sep 17 00:00:00 2001 > From: Darshit Shah > Date: Sat, 1 Nov 2014 05:34:04 +0530 > Subject: [PATCH] Remove extra padding from the progress bar > > --- > src/ChangeLog | 7 +++ > src/progress.c | 7 ++- > 2 files c

Re: [Bug-wget] [Bug-Wget] Policy on commit messages

2014-11-01 Thread Giuseppe Scrivano
Mike Frysinger writes: > On 01 Nov 2014 14:05, Tim Rühsen wrote: >> Am Samstag, 1. November 2014, 05:43:11 schrieb Darshit Shah: >> > In the last few days we've seen a huge flurry of activity and a large >> > number >> > of patches have been committed to the code base. As someone who was >> > tr

Re: [Bug-wget] [PATCH] Stylistic and idiomatic cleanups in Perl tests

2014-11-01 Thread Darshit Shah
On 11/01, Pär Karlsson wrote: Sorry, my bad. :-/ Amended in new patch attached. Pushed. Thanks for the contribution! -- Thanking You, Darshit Shah pgpvS1LXupKcE.pgp Description: PGP signature

Re: [Bug-wget] [PATCH 1/5] use pkg-config to check zlib deps if available

2014-11-01 Thread Darshit Shah
On 11/01, Giuseppe Scrivano wrote: Darshit Shah writes: On 10/31, Mike Frysinger wrote: On 31 Oct 2014 16:50, Giuseppe Scrivano wrote: Mike Frysinger writes: > Newer versions of these packages ship with pkg-config files, so if we can > detect it via those, do so. If that fails, fall back t

Re: [Bug-wget] [PATCH 1/5] use pkg-config to check zlib deps if available

2014-11-01 Thread Giuseppe Scrivano
Darshit Shah writes: > On 10/31, Mike Frysinger wrote: >>On 31 Oct 2014 16:50, Giuseppe Scrivano wrote: >>> Mike Frysinger writes: >>> > Newer versions of these packages ship with pkg-config files, so if we can >>> > detect it via those, do so. If that fails, fall back to the old methods. >>> >

Re: [Bug-wget] Features request

2014-11-01 Thread taras.malivanc...@totaldefense.com
Darshit Shah wrote: On 11/01, taras.malivanc...@totaldefense.com wrote: a case, what would be the expected outcome? Should Wget delete the larger file? Or retain it? It should delete partial file.The full file must not be downloaded , the download will stop when downloaded stuff reaches limi

Re: [Bug-wget] [PATCH 5/5] use pkg-config to check gnutls deps if available

2014-11-01 Thread Mike Frysinger
On 01 Nov 2014 14:11, Darshit Shah wrote: > If we're to merge these patches, all of them should be squashed into one > since > they all perform the same action. I've merged all the patches into a single > one > and added ChangeLog entry to its right place. If others agree, I'll push this > pat

Re: [Bug-wget] [Bug-Wget] Policy on commit messages

2014-11-01 Thread Mike Frysinger
On 01 Nov 2014 14:05, Tim Rühsen wrote: > Am Samstag, 1. November 2014, 05:43:11 schrieb Darshit Shah: > > In the last few days we've seen a huge flurry of activity and a large number > > of patches have been committed to the code base. As someone who was > > traveling and hence unable to keep up w

Re: [Bug-wget] [Bug-Wget] Policy on commit messages

2014-11-01 Thread Darshit Shah
On 11/01, Tim Rühsen wrote: Am Samstag, 1. November 2014, 05:43:11 schrieb Darshit Shah: Hi, In the last few days we've seen a huge flurry of activity and a large number of patches have been committed to the code base. As someone who was traveling and hence unable to keep up with the mailing li

Re: [Bug-wget] [PATCH 5/5] use pkg-config to check gnutls deps if available

2014-11-01 Thread Darshit Shah
On 11/01, Tim Rühsen wrote: Am Samstag, 1. November 2014, 14:11:53 schrieb Darshit Shah: I'm in favour of using pkg-config for detection of these libraries and would like to see the manual checking code gone in a future release. If we're to merge these patches, all of them should be squashed in

Re: [Bug-wget] [Bug-Wget] Policy on commit messages

2014-11-01 Thread Daniel Stenberg
On Sat, 1 Nov 2014, Tim Rühsen wrote: If we move to detailed commit messages (I would appreciate it), then we should autogenerate ChangeLog files from the commit messages. You will find many projects out there already doing so. I would strongly support this as well. -- / daniel.haxx.se

Re: [Bug-wget] [PATCH 5/5] use pkg-config to check gnutls deps if available

2014-11-01 Thread Tim Rühsen
Am Samstag, 1. November 2014, 14:11:53 schrieb Darshit Shah: > I'm in favour of using pkg-config for detection of these libraries and would > like to see the manual checking code gone in a future release. > > If we're to merge these patches, all of them should be squashed into one > since they all

Re: [Bug-wget] [Bug-Wget] Policy on commit messages

2014-11-01 Thread Tim Rühsen
Am Samstag, 1. November 2014, 05:43:11 schrieb Darshit Shah: > Hi, > > In the last few days we've seen a huge flurry of activity and a large number > of patches have been committed to the code base. As someone who was > traveling and hence unable to keep up with the mailing lists, I tried to > cat

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

2014-11-01 Thread Giuseppe Scrivano
Darshit Shah writes: > On 10/31, Giuseppe Scrivano wrote: >>Darshit Shah writes: >> >>> Ever since the Perl tests have been ported to the parallel test >>> harness, Test-proxied-https-auth.px has always failed for me. In a >>> prior thread, we'd identified the problem as a race between the HTTPS

Re: [Bug-wget] [PATCH] Stylistic and idiomatic cleanups in Perl tests

2014-11-01 Thread Pär Karlsson
Sorry, my bad. :-/ Amended in new patch attached. /Pär 2014-11-01 7:00 GMT+01:00 Darshit Shah : > On 11/01, Darshit Shah wrote: > >> On 10/31, Pär Karlsson wrote: >> >>> Thanks for the tips, I was going crazy trying to make git to send the >>> emails via git send-email, and it just ended up weir

Re: [Bug-wget] [PATCH 5/5] use pkg-config to check gnutls deps if available

2014-11-01 Thread Darshit Shah
I'm in favour of using pkg-config for detection of these libraries and would like to see the manual checking code gone in a future release. If we're to merge these patches, all of them should be squashed into one since they all perform the same action. I've merged all the patches into a single