On Wed, Mar 24, 2010 at 04:20:53PM -0300, Luiz Capitulino wrote: > On Tue, 23 Mar 2010 11:27:54 +0100 > Markus Armbruster <arm...@redhat.com> wrote: > > > PATCH 3/4 changes syntax of set_link's second argument from up|down to > > on|off. I feel that the argument needs to be boolean in QMP, and this > > is the simplest way to get it. > > > > Alternatives I could try if the syntax change is unwanted: > > > > * Use the old string argument in QMP. Easy. > > > > * Don't convert set_link, create a new command with a boolean > > argument. > > > > * Create a argument parser for up|down. > > I like your approach. Daniel do you use set_link in libvirt already? > I've grepped around I didn't found any reference for it.
We don't currently use it, but plan to in the not too distant future. Regards, Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://deltacloud.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|