[libvirt] [PATCH 1/3] virt-xml-validate: add --help/--version option

2013-08-19 Thread Eric Blake
All good tools should have --help and --version output :) * tools/virt-xml-validate.in: Add option parsing. Output errors to stderr. Update documentation to match. * tools/Makefile.am (virt-xml-validate): Substitute version. Signed-off-by: Eric Blake ebl...@redhat.com --- tools/Makefile.am

Re: [libvirt] [PATCH 1/3] virt-xml-validate: add --help/--version option

2013-08-19 Thread Eric Blake
On 08/19/2013 04:43 PM, Eric Blake wrote: All good tools should have --help and --version output :) * tools/virt-xml-validate.in: Add option parsing. Output errors to stderr. Update documentation to match. * tools/Makefile.am (virt-xml-validate): Substitute version. + --) shift ;; +

Re: [libvirt] [PATCH 1/3] virt-xml-validate: add --help/--version option

2013-08-19 Thread Guan Nan Ren
ACK - Original Message - From: Eric Blake ebl...@redhat.com To: libvir-list@redhat.com Sent: Monday, August 19, 2013 6:43:13 PM Subject: [libvirt] [PATCH 1/3] virt-xml-validate: add --help/--version option All good tools should have --help and --version output :) * tools/virt-xml

Re: [libvirt] [PATCH 1/3] virt-xml-validate: add --help/--version option

2013-08-19 Thread Guan Nan Ren
- Original Message - From: Eric Blake ebl...@redhat.com Cc: libvir-list@redhat.com Sent: Monday, August 19, 2013 6:58:56 PM Subject: Re: [libvirt] [PATCH 1/3] virt-xml-validate: add --help/--version option On 08/19/2013 04:43 PM, Eric Blake wrote: All good tools should have --help

Re: [libvirt] [PATCH 1/3] virt-xml-validate: add --help/--version option

2013-08-19 Thread Eric Blake
On 08/19/2013 07:51 PM, Guan Nan Ren wrote: This treats '-' as an unrecognized option (which is a bit misleading, since in POSIX terminology a lone - is NOT an option but an argument). It also raises the question - should we allow for an XMLFILE of '-' to mean reading from stdin, the way POSIX