On Sat, Dec 15, 2018 at 07:53:22AM -0600, Eric Blake wrote: > @@ -557,13 +660,14 @@ int main(int argc, char **argv) > Error *local_err = NULL; > BlockdevDetectZeroesOptions detect_zeroes = > BLOCKDEV_DETECT_ZEROES_OPTIONS_OFF; > QDict *options = NULL; > - const char *export_name = ""; /* Default export name */ > + const char *export_name = NULL;
You might want to comment that export_name is now a tri-state value, with NULL meaning that it's not set by the user. Nevertheless the patch looks good, and the addition of the example (including host name setting) resolves my previous problem with the new functionality. Therefore: Reviewed-by: Richard W.M. Jones <rjo...@redhat.com> Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-builder quickly builds VMs from scratch http://libguestfs.org/virt-builder.1.html
