[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-wget] Marking Release v1.17.1?
From: |
Giuseppe Scrivano |
Subject: |
Re: [Bug-wget] Marking Release v1.17.1? |
Date: |
Fri, 11 Dec 2015 10:03:57 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Jernej Simončič <jernej|address@hidden> writes:
> On Tuesday, December 8, 2015, 21:45:31, Darshit Shah wrote:
>
>> With my last set of patches, we have fixed all the issues reported /
>> identified
>> after the 1.17 release. Hence, maybe we should consider releasing a 1.17.1
>> bugfix release?
>
> Sorry about reporting this late, it forgot about it a bit: when I was
> preparing my Windows release, I noticed that metalink support first
> didn't want to compile (due to missing sys/errno.h - just #ifdefing it
> out didn't seem to break anything), and then that hash validation
> failed, because the file was being opened in text mode. I did a quick
> and dirty fix for my build
> <https://eternallybored.org/misc/wget/src/wget-1.17-win32-metalink.patch>,
> but this probably isn't correct for non-Windows systems.
Thanks for the patch, one piece is already present upstream, I will
commit the missing part:
>From bf56bf4560cef3c1591487e7df1f2cb5e5ad0303 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jernej=20Simon=C4=8Di=C4=8D?=
<jernej|address@hidden>
Date: Fri, 11 Dec 2015 09:58:30 +0100
Subject: [PATCH] * src/metalink.c: Specify 'rb' as mode to open file
---
src/metalink.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/metalink.c b/src/metalink.c
index 986303d..25737b3 100644
--- a/src/metalink.c
+++ b/src/metalink.c
@@ -168,7 +168,7 @@ retrieve_from_metalink (const metalink_t* metalink)
FILE *local_file;
/* Check the digest. */
- local_file = fopen (filename, "r");
+ local_file = fopen (filename, "rb");
if (!local_file)
{
logprintf (LOG_NOTQUIET, _("Could not open downloaded
file.\n"));
--
2.5.0
> Another thing that would be nice to include is the taskbar progressbar
> patch:
> <https://eternallybored.org/misc/wget/src/taskbar-progress.patch>
> This one is based on patch by Ángel González and tbprogress.c from
> Gisle Vanem.
Darshit, is there anything left on the progress bar?
I will probably tag 1.17.1 later today.
Regards,
Giuseppe
- [Bug-wget] Marking Release v1.17.1?, Darshit Shah, 2015/12/08
- Re: [Bug-wget] Marking Release v1.17.1?, Mike Frysinger, 2015/12/08
- Re: [Bug-wget] Marking Release v1.17.1?, Giuseppe Scrivano, 2015/12/09
- Re: [Bug-wget] Marking Release v1.17.1?, Jernej Simončič, 2015/12/10
- Re: [Bug-wget] Marking Release v1.17.1?,
Giuseppe Scrivano <=
- Re: [Bug-wget] Marking Release v1.17.1?, Jernej Simončič, 2015/12/11
- Re: [Bug-wget] Marking Release v1.17.1?, Gisle Vanem, 2015/12/12
- Re: [Bug-wget] Marking Release v1.17.1?, Jernej Simončič, 2015/12/12
- Re: [Bug-wget] Marking Release v1.17.1?, Eli Zaretskii, 2015/12/13
- Re: [Bug-wget] Marking Release v1.17.1?, Giuseppe Scrivano, 2015/12/16
- Re: [Bug-wget] Marking Release v1.17.1?, Eli Zaretskii, 2015/12/16
- Re: [Bug-wget] Marking Release v1.17.1?, Gisle Vanem, 2015/12/16
- Re: [Bug-wget] Marking Release v1.17.1?, Eli Zaretskii, 2015/12/16
- Re: [Bug-wget] Marking Release v1.17.1?, Giuseppe Scrivano, 2015/12/16