[pve-devel] applied: [PATCH qemu-server] fix #1570: fix template backup with pigz

Fabian Grünbichler f.gruenbichler at proxmox.com
Wed Dec 6 14:56:10 CET 2017


On Mon, Dec 04, 2017 at 08:38:09AM +0100, Thomas Lamprecht wrote:
> On 12/01/2017 11:43 AM, Dominik Csapak wrote:
> > when using pigz the resulting commandline would look like:
> > 
> > pigz -p 4>file
> > 
> > which resulted in pigz erroring out because it got no parameter for -p
> > (because the shell interpreted the 4>file as a file descriptor)
> > 
> > this patch adds a space so that the resulting line is
> > pigz -p 4 > file
> > 
> > Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> > ---
> >  PVE/VZDump/QemuServer.pm | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/PVE/VZDump/QemuServer.pm b/PVE/VZDump/QemuServer.pm
> > index a630829..42680f8 100644
> > --- a/PVE/VZDump/QemuServer.pm
> > +++ b/PVE/VZDump/QemuServer.pm
> > @@ -274,7 +274,7 @@ sub archive {
> >  	    $outcmd = "exec:cat";
> >  	}
> >  
> > -	$outcmd .= ">$filename" if !$opts->{stdout};
> > +	$outcmd .= " > $filename" if !$opts->{stdout};
> >  
> >  	my $cmd = ['/usr/bin/vma', 'create', '-v', '-c', $conffile];
> >  	push @$cmd, '-c', $firewall if -e $firewall;
> > 
> 
> Reviewed-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel




More information about the pve-devel mailing list