Thanks for doing this Brad. These changes look fine to me. I'm looking 
to make some changes in the same area of testutils as you are. I think 
both our changes can go in without problems, but if you'd check the 
webrev for 4570 to double check me, I'd appreciate it.

Thanks,
Brock

Brad Hall wrote:
> Hi,
>
> Link to webrev: http://cr.opensolaris.org/~bhall/figleaf/
>
> More information on figleaf:
> http://darcs.idyll.org/~t/projects/figleaf/doc/
>
> Figleaf is a tool for python code coverage analysis.
>
> This adds a new makefile target "coverage" which will run the test suite
> under figleaf.  If figleaf isn't installed, it will install it.  When it
> is done running it generates a coverage report in html format.
>
> Unfortunately it's not really easy to get the output into a reasonable
> text format at this point.  Doing that will require some modifications
> to figleaf.  (It can output to coverage.py format, but it doesn't give
> any control over where the files go, names, etc., currently.)
>
> Thanks,
> Brad
> _______________________________________________
> pkg-discuss mailing list
> [email protected]
> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
>   

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to