#7716: sage -coverage enhancement
-----------------------------+----------------------------------------------
Reporter: roed | Owner: mvngu
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: documentation | Keywords: coverage
Work_issues: | Upstream: N/A
Reviewer: | Author: roed
Merged: | Dependencies:
-----------------------------+----------------------------------------------
Comment(by jhpalmieri):
Some issues:
- sage-coverageall doesn't deal with unrecognized options well, since it
just parses the output from sage-coverage. So if you pass an unrecognized
option, it ends up saying "No files in ...". Instead of doing
{{{
r = os.popen('sage -coverage %s * | grep SCORE'%opt).readlines()
}}}
we should probably set P to be the process and check its return status
before asking for its output.
- the options should work with either one or two hyphens.
- We should have a "--help" option for sage-coverage which does the same
thing as running sage-coverage with no arguments: print the usage. I
think we should also expand this usage message.
I'm attaching a "diff" which makes these changes. I haven't looked at the
rest of the code in detail yet, but I may soon.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7716#comment:12>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
