Re: [libvirt] [PATCH] virsh: allow empty string arguments

2011-03-15 Thread Eric Blake
On 03/15/2011 02:13 PM, Jiri Denemark wrote: > On Mon, Mar 14, 2011 at 10:46:07 -0600, Eric Blake wrote: >> "virsh connect ''" should try to connect to the default connection, >> but the previous patch made it issue a warning about an invalid URI. >> >> * tools/virsh.c (VSH_OFLAG_EMPTY_OK): New opt

Re: [libvirt] [PATCH] virsh: allow empty string arguments

2011-03-15 Thread Jiri Denemark
On Mon, Mar 14, 2011 at 10:46:07 -0600, Eric Blake wrote: > "virsh connect ''" should try to connect to the default connection, > but the previous patch made it issue a warning about an invalid URI. > > * tools/virsh.c (VSH_OFLAG_EMPTY_OK): New option flag. > (vshCommandOptString): Per the declara

[libvirt] [PATCH] virsh: allow empty string arguments

2011-03-14 Thread Eric Blake
"virsh connect ''" should try to connect to the default connection, but the previous patch made it issue a warning about an invalid URI. * tools/virsh.c (VSH_OFLAG_EMPTY_OK): New option flag. (vshCommandOptString): Per the declaration, value is required to be non-NULL. Honor new flag. (opts_conne