On Fri, Jan 10, 2014 at 01:33:38PM +0000, Richard W.M. Jones wrote: > The API is now pretty confusing. Each OBool flag is really a > tristate. It can either be "true", "false" or "not specified". > > Therefore I think it should be: > > xattributes:true # copies only xattrs and nothing else > all:true # copies everything > all:true xattributes:false # copies everything except xattrs > > In other words, 'all' changes the default (ie. "not specified") state > of the other flags. > > To be clearer, the four OBool parameters would be: > > [OBool "all"; OBool "mode"; OBool "ownership"; OBool "xattributes"]
So looking at your v2 patch a bit more, I think this is what you already did, except that "skipmode" is backwards from the other flags. I think we're in agreement, except I think "skipmode" should just become "mode" and not have negated meaning compared to the other flags. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones 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