On Fri 24 Apr 2009 at 02:37PM, Brock Pytlik wrote:
> Webrev:
> http://cr.opensolaris.org/~bpytlik/ips-5247-v1/
>
> Bug:
> 5247 Test suite should provide timing info
>
> http://defect.opensolaris.org/bz/show_bug.cgi?id=5247
>
> This adds a -t option to the setup.py test command. When it's present,
> it produces a timing_info.txt file in the tests directory which has info
> for the test suite classes and functions. There's lots more work that
> could be done to make it fancier and have it present nicer output, but
> for now, it's functional and produces apparently correct output. So I'd
> generally like to leave other functionality requests for future RFEs.
>
> The reason that -t is a flag, and not an option for a file is bug 8397.
Should emitting timings be the default behaviour instead of an
optional one?
Also, I would find it easier to read if you didn't use quite so
many decimal places :) I think 3.55 is more helpful than 3.55107688904.
-dp
--
Daniel Price, Solaris Kernel Engineering http://blogs.sun.com/dp
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
