On Mon, Mar 12, 2018 at 09:01:57AM +0200, Mykola Ivanets wrote:
> Rebased.
> 
> The only thing left unclear is: "Would you like to document behavior for 
> non-MBR partitions
> more clearly or keep previous implementation despite MBR partition type 
> doesn't make sense
> for non-MBR partitions at all?"

https://gb.redhat.com/archives/libguestfs/2018-February/msg00027.html

I don't agree that the existing implementation doesn't make sense for
GPT.  It returns "primary", but the reimplementation returns an error.
That's going to break callers who weren't careful about only using
this API on MBR, and I think that is unnecessary breakage.  GPT
partitions are all "primary".

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
libguestfs lets you edit virtual machines.  Supports shell scripting,
bindings from many languages.  http://libguestfs.org

_______________________________________________
Libguestfs mailing list
Libguestfs@redhat.com
https://www.redhat.com/mailman/listinfo/libguestfs

Reply via email to