[Libguestfs] [PATCH] builder: Remove duplicate planner transition.
Pino Toscano
ptoscano at redhat.com
Tue Oct 6 12:00:12 UTC 2015
On Tuesday 06 October 2015 11:22:49 Richard W.M. Jones wrote:
> Seems to be a copy and paste bug in
> commit 62cc7d3361127b4e007f8e23028213852be09124.
> ---
> builder/builder.ml | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/builder/builder.ml b/builder/builder.ml
> index af9c2f5..6850f8f 100644
> --- a/builder/builder.ml
> +++ b/builder/builder.ml
> @@ -456,10 +456,9 @@ let main () =
> * Don't run 'qemu-img resize' on an auto format. This is to
> * force an explicit conversion step to a real format.
> *)
> - else if output_size > old_size && is_not `Template && List.mem_assoc `Format itags then (
> + else if output_size > old_size && is_not `Template
> + && List.mem_assoc `Format itags then
> tr `Disk_resize 60 ((`Size, Int64.to_string output_size) :: itags);
> - tr `Disk_resize 60 ((`Size, Int64.to_string output_size) :: itags);
> - );
>
> (* qemu-img convert is always possible, and quicker. It doesn't
> * resize, but it does change the format.
LGTM.
Thanks,
--
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20151006/34f00a95/attachment.sig>
More information about the Libguestfs
mailing list