Éric Araujo mer...@netwok.org added the comment:
The revision you linked to seems to say that Georg was the one who added this
code.
--
nosy: +georg.brandl
versions: +Python 3.3 -Python 3.1
___
Python tracker rep...@bugs.python.org
Georg Brandl ge...@python.org added the comment:
The revision you linked to seems to say that Georg was the one who
added this code.
Can you elaborate? Line 355 in trunk was last edited by edloper.
--
___
Python tracker rep...@bugs.python.org
Éric Araujo mer...@netwok.org added the comment:
I misread the Subversion viewer, it was actually PJE in r45248 (if I read
correctly this time :)
--
nosy: +pje
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6703
Phillip J. Eby p...@telecommunity.com added the comment:
No, I only consolidated the two copies of the code to a single function, in
order to more easily add the PEP 302 support. The feature was already there.
--
___
Python tracker
Éric Araujo mer...@netwok.org added the comment:
Patch looks good. One error and style suggestions on rietveld.
--
components: +Library (Lib)
nosy: +eric.araujo
stage: - patch review
versions: +Python 3.1 -Python 2.6
___
Python tracker
Chris Withers ch...@simplistix.co.uk added the comment:
What this patch doesn't appear to address is that there doesn't appear to be
any point to module_relative. Can anyone clarify what the intention of this
option is? (it doesn't appear to have any meaningful point from my
experience...)
anatoly techtonik techto...@gmail.com added the comment:
Sure doctests should be crossplatform by default when possible. The patch
should fix the behavior for windows. It also cleans the doc and code a bit.
Can you provide some testcases?
--
keywords: +patch
nosy: +techtonik
Added
anatoly techtonik techto...@gmail.com added the comment:
http://codereview.appspot.com/815042/show
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6703
___
Chris Withers ch...@simplistix.co.uk added the comment:
Hmm, I don't think tests will fail, however, there are cryptic docs for this...
http://docs.python.org/library/doctest.html#basic-api
I don't really get what module_relative is about and I've always run into the
non-
cross-platform issue
New submission from Chris Withers ch...@simplistix.co.uk:
Hi All,
Line 355 of this code on the 2.6 branch and trunk:
http://svn.python.org/view/python/branches/release26-
maint/Lib/doctest.py?view=annotate
http://svn.python.org/view/python/trunk/Lib/doctest.py?annotate=69012
...contain a
Benjamin Peterson benja...@python.org added the comment:
How about removing it and seeing if tests fail?
--
nosy: +benjamin.peterson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6703
___
11 matches
Mail list logo