[Libvirt-cim] [PATCH] [TEST] #3 Add -d to duplicate error output to stderr

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Wed Apr 16 16:17:23 UTC 2008


Zhengang Li wrote:
> zli at 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 at linux.vnet.ibm.com




More information about the Libvirt-cim mailing list