#9739: Handle duplicate file basenames when testing multiple files in parallel
-------------------------------+--------------------------------------------
Reporter: mpatel | Owner: mvngu
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-4.7.1
Component: doctest | Keywords: doctest scripts
Work_issues: | Upstream: N/A
Reviewer: Robert Bradshaw | Author: Mitesh Patel, John Palmieri
Merged: | Dependencies:
-------------------------------+--------------------------------------------
Comment(by jhpalmieri):
Replying to [comment:60 leif]:
> Replying to [comment:58 jhpalmieri]:
> > Oops, just found a mistake. In non-Sage library code, when doctesting
"file0.py", we write a line
> {{{
> from file0 import *
> }}}
> > With the name mangling, this doesn't work anymore: the periods confuse
things, and so would hyphens, commas, and other symbols not allowed in
python module names.
>
> Hence underscores, which I originally thought of?
Well, the host name could contain all sorts of characters in it, couldn't
it? Same for the directories in the path to the file, especially since
we're talking about files not in the Sage library. Doing some sort of
regexp search and replace is a lot of work for perhaps minimal gain. It
certainly doesn't have to do with the issue on this ticket.
> It doesn't make sense to copy [only] each single non-library file to
doctest to the temporary directory anyway, as it might import other files
located in the original directory.
Outside the scope of this ticket. If we leave that part alone, we're not
creating a new bug, just leaving a less-than-perfect implementation in
place.
Meanwhile, if you want to add in some print statements, comments, some of
the relevant parts of Mitesh's patch, or anything else, go ahead. I have
to work on some other things for at least a few days.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9739#comment:61>
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.
