[libvirt] [PATCH] virt-xml-validate: Allow input to be read from stdin
Johannes Holmberg
johannes.holmberg at dataductus.se
Wed Jun 24 16:17:35 UTC 2015
----- On 24 Jun, 2015, at 09:14, Martin Kletzander mkletzan at redhat.com wrote:
> Why don't you just set XMLFILE="$TMPFILE" ? That would get rid of lot
> of the code changes below and would be more readable.
>
> Other than that it looks good.
Good question, with a somewhat long answer. I initially did exactly what you suggest, and yes, it makes for simpler code. The only problem with it is that when xmllint finds a problem with the xml, it will output a string on the format "file name:line number: error description". If the file was provided on stdin, the file name will be something like "/tmp/virt-xml.asdf" which will not make much sense to the user.
So instead I tried to be consistent: if virt-xml-validate received a filename, xmllint will receive a filename. If virt-xml-validate received data on stdin, xmllint will receive data on stdin.
This way, when the xml is provided on stdin and xmllint finds a problem, the file name in the error message will just show up as "-", which is probably more in line with what the user was expecting.
Anyway, it's not something I feel super-strongly about, so if you prefer to have it changed, just let me know and I'll have an updated patch for you.
/Johannes
More information about the libvir-list
mailing list