Proposal for small addition to testsuite output

[WENJI.HUANG@ORACLE.COM]

> Wenji Huang <wenji.huang@oracle.com> writes:
> 
> > According to Kris' proposal, I made a small change to systemtap.exp. [...]
> > +proc get_system_info {} {
> 
> This part is OK
> 
> > +get_system_info
> > +set logname "$logname\nHost: $Host\nSnapshot: $Snapshot\nDistro: $Distro\n"
> 
> I suspect that changing "logname" like this is suspect.  How
> about
> simply computing and printing the new fields in print_systemtap_version,
> which is already defined in testsuite/lib/systemtap.exp.  They
> should be
> changed to log the data too, not just print it to the screen.

Yes, I tried the similar way at the beginning.  But found that this info can't be put on the very top of log file.

Not as Kris' preference.  Don't know it appropriate for auto-test system ?

> 
> - FChE

Thanks,
Wenji
-------------- next part --------------
A non-text attachment was scrubbed...
Name: systemtap.log
Type: application/octet-stream
Size: 847 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/systemtap/attachments/20070901/8a793942/attachment.obj>