bug-wget
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] [PATCH 12/25] New test: --continue shall keep fully retri


From: Giuseppe Scrivano
Subject: Re: [Bug-wget] [PATCH 12/25] New test: --continue shall keep fully retrieved Metalink files (HTTP 416)
Date: Sun, 11 Sep 2016 22:51:27 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Hi Matthew,

Matthew White <address@hidden> writes:

> [Coverity Scan is ok, make syntax-check is ok, make check-valgrind is ok, 
> contrib/check-hard is ok]
>
> This introduces a new Metalink test to verify that when --continue is used, 
> after a HTTP 416 answer a file that fails a sanity test isn't renamed/removed.
>
> The following description is verbatim from the patch:
> -----
> Ensure that --continue doesn't rename/remove existing and/or fully
> retrieved files which fail the sanity tests.
> -----

can this test be included together with the patch that introduces this change?

> +MetaXml = re.sub (r'{{FILE1_HASH}}', File1_sha256, MetaXml)
> +MetaXml = re.sub (r'{{FILE2_HASH}}', File2_sha256, MetaXml)
> +MetaXml = re.sub (r'{{FILE3_HASH}}', File3_sha256, MetaXml)
> +MetaXml = re.sub (r'{{FILE4_HASH}}', File4_sha256, MetaXml)
> +MetaXml = re.sub (r'{{FILE5_HASH}}', File5_sha256, MetaXml)
> +MetaXml = re.sub (r'{{SRV_HOST}}', srv_host, MetaXml)
> +MetaXml = re.sub (r'{{SRV_PORT}}', str (srv_port), MetaXml)
> +MetaFile.content = MetaXml

would just MetaXml.replace be enough here as well?

Thanks,
Giuseppe



reply via email to

[Prev in Thread] Current Thread [Next in Thread]