Re: [libvirt] [PATCH 7/8] Fix up inconsistent virsh option error reporting.

2010-07-19 Thread Chris Lalancette
On 07/17/10 - 06:08:03PM, Matthias Bolte wrote: > 2010/7/8 Chris Lalancette : > > The virsh option error reporting was not being used > > consistently; some commands would spit out errors on > > missing required options while others would just silently fail. > > However, vshCommandOptString knows w

Re: [libvirt] [PATCH 7/8] Fix up inconsistent virsh option error reporting.

2010-07-17 Thread Matthias Bolte
2010/7/8 Chris Lalancette : > The virsh option error reporting was not being used > consistently; some commands would spit out errors on > missing required options while others would just silently fail. > However, vshCommandOptString knows which ones are required > and which ones aren't, so make it

[libvirt] [PATCH 7/8] Fix up inconsistent virsh option error reporting.

2010-07-08 Thread Chris Lalancette
The virsh option error reporting was not being used consistently; some commands would spit out errors on missing required options while others would just silently fail. However, vshCommandOptString knows which ones are required and which ones aren't, so make it spit out an error where appropriate.