Re: [Libvirt-cim] [PATCH] [TEST] #3 Add -d to duplicate error output to stderr
Zhengang Li wrote:
zli(a)linux.vnet.ibm.com wrote:
> .#3:
> - Fix the bug that ignore bugs if no -d is specified.
> @@ -119,6 +134,7 @@ def do_main(types=['Xen'], p=parser):
> else:
> def do_try():
> try:
> + log_param(options.debug==True)
This is the only line .#3 changed.
The parser doesn't assign a 'False' value to options.debug if no -d is
specified.
So we have three case:
1. log_param() in test case, debug is 'None'
2. log_param(debug) in do_try(), and -d is specified, debug is 'True'
3. log_param(debug) in do_try(), and no -d is specified, debug is 'False'
Excellent - thanks Zhengang! I've pushed this in. Sorry to make you
re-work this so many times.
I think this will be handy when running a single test. I use cimtest
for spot checking things here and there, so having the option to print
the log will save me a few keystrokes. =)
I hope others find it useful.
--
Kaitlin Rupert
IBM Linux Technology Center
kaitlin(a)linux.vnet.ibm.com