#4706: fix race condition between doctest creation and running
------------------------+---------------------------------------------------
Reporter: mabshoff | Owner: mabshoff
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: doctest | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
------------------------+---------------------------------------------------
Changes (by mderickx):
* status: new => needs_review
* upstream: => N/A
Old description:
> With high -tp numbers (i.e. 16) on sage.math one will see similar issues
> to the one below:
> {{{
> sage -t -long devel/sage/sage/libs/symmetrica/symmetrica.pyx
> File "/scratch/mabshoff/release-
> cycle/sage-3.2.alpha0/tmp/.doctest_symmetrica.py", line 29
> >>> test_integer(Integer(1))###line 539:_sage_ >>> test_integer(1)
> ^
> IndentationError: unexpected indent
> }}}
>
> This is likely a race condition between creating the file and the actual
> running of the doctest. The issue is not specific to -tp.
>
> A potential solution might be to create all .doctest_$FOO files and then
> start running them. This might also fix the problem with
> {{{
> sage -t -long devel/sage/sage/symbolic/constants.pyx
> [0.1 s]
> }}}
> in Sage 3.2.1+ which is caused by no doctests being executed since (a)
> either there are no doctests in that file or (b) we are running optional
> doctests only.
>
> Cheers,
>
> Michael
New description:
With high -tp numbers (i.e. 16) on sage.math one will see similar issues
to the one below:
{{{
sage -t -long devel/sage/sage/libs/symmetrica/symmetrica.pyx
File "/scratch/mabshoff/release-
cycle/sage-3.2.alpha0/tmp/.doctest_symmetrica.py", line 29
>>> test_integer(Integer(1))###line 539:_sage_ >>> test_integer(1)
^
IndentationError: unexpected indent
}}}
This is likely a race condition between creating the file and the actual
running of the doctest. The issue is not specific to -tp.
A potential solution might be to create all .doctest_$FOO files and then
start running them. This might also fix the problem with
{{{
sage -t -long devel/sage/sage/symbolic/constants.pyx
[0.1 s]
}}}
in Sage 3.2.1+ which is caused by no doctests being executed since (a)
either there are no doctests in that file or (b) we are running optional
doctests only.
Cheers,
Michael
don't apply anything!
--
Comment:
I think this is an old ticket which can be closed and has been fixed in
the mean time. At least during the parralel builds I did (and I did quite
a few this sage days already) I never saw this message.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4706#comment:2>
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.
