#8680: untabify Sage again, and make it stick
------------------------------+---------------------------------------------
Reporter: jhpalmieri | Owner: jason
Type: enhancement | Status: positive_review
Priority: major | Milestone: sage-4.5
Component: doctest | Keywords:
Author: John Palmieri | Upstream: N/A
Reviewer: David Loeffler | Merged:
Work_issues: |
------------------------------+---------------------------------------------
Comment(by leif):
Sorry for the delay, I'm currently trying to make sense of all the
doctest-related scripts... ;-) (By doing so, I encountered a number of
flaws, inconsistencies and things I'm not sure if they are correct and/or
intentional.)
I suggest splitting this ticket into two, such that the tab-removing Sage
library patch can be merged immediately (cf. Robert Miller's mail, not
posted on sage-release).
I don't think the detection of "illegal" tabs is in the right place/done
the appropriate way; it's IMHO undesirable to slow down ''every''
doctesting by scanning each file twice in addition.
An alternative would be to (also) make tab-checking an ''option'' to the
test scripts and/or control it by another environment variable, something
I wanted to provide a reviewer patch for, but couldn't yet because of the
issues mentioned above that ate my time. :(
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8680#comment:18>
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.
