[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