[pve-devel] applied: [PATCH v9 common 2/2] tools: download_file_from_url: move check for existing file outside eval
Thomas Lamprecht
t.lamprecht at proxmox.com
Wed Jun 16 12:46:08 CEST 2021
On 16.06.21 11:35, Lorenz Stechauner wrote:
> it is not necessary to include this block in the eval which when it
> fails tries to unlink $tmpdest, because in the check for the existing
> file $tmpdest is not used.
> ---
> src/PVE/Tools.pm | 26 +++++++++++++-------------
> 1 file changed, 13 insertions(+), 13 deletions(-)
>
applied, thanks! FYI I made a not directly related followup (send to the list).
More information about the pve-devel
mailing list