Chris Jerdonek added the comment:
> I think we generally upgrade it exactly when there is a new Sphinx feature we
> want for our docs :)
:) Would the appropriate way to handle it be then to create an issue to
"upgrade Sphinx when XXX issue is resolved" and link to the corre
Chris Jerdonek added the comment:
I created a Sphinx issue for this here:
https://bitbucket.org/birkenfeld/sphinx/issue/996/expose-glossary-entry-link-on-hover
--
___
Python tracker
<http://bugs.python.org/issue15
Chris Jerdonek added the comment:
I created a Sphinx issue for this here:
https://bitbucket.org/birkenfeld/sphinx/issue/997/index-targets-not-getting-created-in
--
___
Python tracker
<http://bugs.python.org/issue15
Chris Jerdonek added the comment:
Attaching patch. I also added two index entries for "file object" and made a
couple wording adjustments to the intro paragraph of the io module.
--
keywords: +patch
stage: -> patch review
versions: +Python 3.2
Added file: http://bu
Chris Jerdonek added the comment:
Good suggestion, David. Here is such sample test code. It is adapted from the
sample code for "ValueError: generator already executing" included in PEP 255:
def test_gen(call_gen_method):
def gen():
call_gen_method(me)
yiel
Chris Jerdonek added the comment:
Thanks. Could either one of you commit this for me?
--
___
Python tracker
<http://bugs.python.org/issue15694>
___
___
Python-bug
Chris Jerdonek added the comment:
Thank you, Andrew. Here is a patch updated with that change (and also merging
with tip).
--
Added file: http://bugs.python.org/file26899/issue-15595-4.patch
___
Python tracker
<http://bugs.python.org/issue15
Chris Jerdonek added the comment:
It might be worth mentioning and/or linking to some of the branching tips
discussed in the FAQ. For example, using `hg share` instead of `hg clone`
eliminates the need to pull between repositories/working directories:
http://docs.python.org/devguide
Chris Jerdonek added the comment:
> I think the aim of that part of the devguide is to give one clear, simple,
> working way to operate on different branches at the same time.
The change can be as simple as adding a sentence along the lines of, "Also see
the version control section
Chris Jerdonek added the comment:
I would be happy to take a look at this and propose a patch.
--
nosy: +cjerdonek
___
Python tracker
<http://bugs.python.org/issue15
Chris Jerdonek added the comment:
I haven't been able to test this via Python because my system sqlite3 version
isn't new enough. But I was able to test this against sqlite3 directly. I
suspect there may be no issue.
John, have you tried naming your constraint?
http://www.
Chris Jerdonek added the comment:
> When building CPython yourself, it will use what version is found on your
> system.
Ned, this is somewhat off-topic to the issue, but do you know of any ways to
tell CPython to use a particular version of sqlite3 at compile time if a system
has dif
Changes by Chris Jerdonek :
--
resolution: works for me -> invalid
stage: needs patch -> committed/rejected
___
Python tracker
<http://bugs.python.org/i
Chris Jerdonek added the comment:
I randomly ran into this issue again. I'm not sure this was ever resolved
(i.e. I think it may always have been different from issue 15111).
I still get the above behavior in the default branch.
And here is what I get in the 3.2 branch (the error inform
Chris Jerdonek added the comment:
I ran into this again because an error while running `./python.exe -m test` was
getting masked. The use of __main__.py in the package may be the
distinguishing characteristic.
--
___
Python tracker
<h
Changes by Chris Jerdonek :
--
keywords: +3.2regression
___
Python tracker
<http://bugs.python.org/issue15316>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Jerdonek added the comment:
Should this issue be fixed before the release? If it is not fixed, certain
problems found after the release may become harder to report and diagnose
(because the true source of error will be masked).
Two months ago issue 15111 which was thought to be the
Chris Jerdonek added the comment:
Here is a formal unit test case that passes in 3.2 but not in 3.3 (a "simpler"
case not using __main__.py).
(script_helper.create_empty_file() doesn't seem to be available in 3.2.)
--
keywords: +patch
Added file: http://bugs.python.org/
Chris Jerdonek added the comment:
You're welcome, Brett. I'll let you or someone else recast the test using the
latest preferred techniques. I was just using the style of the immediately
surrounding tests.
--
___
Python trac
Changes by Chris Jerdonek :
--
nosy: +cjerdonek
___
Python tracker
<http://bugs.python.org/issue15767>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Jerdonek added the comment:
Hi moijes12, for instructions on compiling and submitting patches, etc., see
the Python Developer's Guide (aka the devguide). You may also be interested in
the Python Mentors Group: http://pythonmentors.com/
Lastly (a small point), in the future you can
Changes by Chris Jerdonek :
--
nosy: +cjerdonek
___
Python tracker
<http://bugs.python.org/issue11664>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Jerdonek added the comment:
Adding Éric because of the interest in test setup and tear down in issue 11664.
--
nosy: +eric.araujo
___
Python tracker
<http://bugs.python.org/issue15
Chris Rebert added the comment:
Revised patch yet again to instead speak of "non-object, non-array" values and
"JSON null, boolean, number, or string" values.
Re: Petri, the patch already mentions the specific parameters one can use to
get stricter behavior, albeit no
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue13212>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue15136>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue14965>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Jerdonek added the comment:
Here is an updated patch for review (just for the default branch for now).
The main change from the previous patch is that the ValueError exception is now
documented.
A few additional comments/questions:
Is there a better way to signify in the documentation
Changes by Chris Jerdonek :
Added file: http://bugs.python.org/file26991/issue-14649-3.patch
___
Python tracker
<http://bugs.python.org/issue14649>
___
___
Python-bug
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue15731>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Jerdonek added the comment:
> Personally I prefer to have the test case create the file(s) used in the test
> dynamically, writing them to the temporary working directory.
I prefer that too, but when I approached this issue, I found that test_doctest
doesn't use unittest. I
Chris Jerdonek added the comment:
Small tweak to a code comment in the patch (`tests` is not itself a
unittest.TestSuite instance).
--
Added file: http://bugs.python.org/file27000/issue-14649-4.patch
___
Python tracker
<http://bugs.python.
Chris Jerdonek added the comment:
Here is a patch that documents and adds tests for the existing behavior (i.e.
keeping the current behavior the same).
I also expanded the patch slightly to cover related edge cases that involve the
interplay between whitespace, empty lines, and indenting
Chris Jerdonek added the comment:
I submitted a patch for this issue on the Sphinx tracker.
--
___
Python tracker
<http://bugs.python.org/issue15693>
___
___
Pytho
Changes by Chris Rebert :
--
keywords: +patch
Added file: http://bugs.python.org/file27008/issue14880.patch
___
Python tracker
<http://bugs.python.org/issue14
Chris Rebert added the comment:
Updated patch. I concur that anyone creating a type whose name has whitespace
in it is just asking for trouble.
--
Added file: http://bugs.python.org/file27009/reprlib.rst.patch
___
Python tracker
<h
New submission from Chris Rebert:
The cross-references, particularly to Popen.communicate(), in the warning box
in the subprocess docs about using Popen.stdout.read() etc. aren't hyperlinked.
The attached patch fixes that.
--
assignee: docs@python
components: Documentation
Chris Rebert added the comment:
So you can. Neat trick.
Patch changed accordingly.
--
Added file: http://bugs.python.org/file27011/link_communicate_warning.patch
___
Python tracker
<http://bugs.python.org/issue15
Chris Rebert added the comment:
Let's try that again.
--
Added file: http://bugs.python.org/file27012/link_communicate_warning.patch
___
Python tracker
<http://bugs.python.org/is
Changes by Chris Rebert :
Removed file: http://bugs.python.org/file27011/link_communicate_warning.patch
___
Python tracker
<http://bugs.python.org/issue15788>
___
___
New submission from Chris Rebert:
To further dissuade people from using subprocess.Popen's shell=True option
unnecessarily, this patch points out that some common shell features are also
available directly in Python via certain modules & functions.
--
assignee: docs@python
c
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue15767>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Rebert added the comment:
Erm, the patch seems to have some ANSI terminal coloring escape code junk (e.g.
"[0;31m") in it...
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.o
Chris Rebert added the comment:
It's been over a year and the patch is trivial. Any chance of it getting
applied soon?
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/is
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue15533>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue7186>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue9650>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue14783>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue14870>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Jerdonek added the comment:
Attached are a few test cases showing that Popen *does* consider cwd when
searching for the executable (as well as for args[0]), and in particular that
you *can* specify the program's path relative to cwd.
I also moved the test_cwd test to be adjacent t
Changes by Chris Jerdonek :
--
nosy: +asvetlov
___
Python tracker
<http://bugs.python.org/issue15533>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Jerdonek added the comment:
Here is a full patch for the default branch (documentation correction and test
cases for the documented behavior).
If this patch looks acceptable, I can prepare a separate patch for 2.7.
--
Added file: http://bugs.python.org/file27018/issue-15533-2
Changes by Chris Jerdonek :
--
keywords: +needs review
stage: -> patch review
___
Python tracker
<http://bugs.python.org/issue15533>
___
___
Python-bugs-lis
Chris Jerdonek added the comment:
python_dir = os.path.dirname(os.path.realpath(sys.executable))
wrong_cwd = os.path.join(python_dir, 'Doc')
Actually, is there a better directory to be using for this? I'd like a
directory that is guaranteed to exist that is in the sa
Chris Jerdonek added the comment:
Here is a new patch that makes no assumptions about the contents of the
directory containing sys.executable.
--
Added file: http://bugs.python.org/file27026/issue-15533-3-default.patch
___
Python tracker
<h
Chris Rebert added the comment:
+1 on including am/pm-related codes. Blame us backwards, non-metric Americans.
Sounds GTM otherwise.
--
___
Python tracker
<http://bugs.python.org/issue9
Chris Jerdonek added the comment:
Updating the patch after discussing with David on IRC.
The two new files are now added to Lib/test instead of to a subdirectory of
Lib/test. Moving the doctest files to a subdirectory can be discussed and
possibly addressed as part of a separate issue
Chris Jerdonek added the comment:
This is a nit, but can you adhere to an 80-character line length? Much (but
not all) of the documentation does.
--
nosy: +cjerdonek
___
Python tracker
<http://bugs.python.org/issue15
Chris Rebert added the comment:
Revised patch in response to comments.
--
Added file: http://bugs.python.org/file27029/subprocess.rst.patch
___
Python tracker
<http://bugs.python.org/issue15
Chris Jerdonek added the comment:
> With your patch 5 applied, test_zipimport_support fails.
You're right. It looks like test_zipimport_support is tightly coupled to
test_doctest. For example--
http://hg.python.org/cpython/file/786d9516663e/Lib/test/test_zipimport_support.py
Chris Jerdonek added the comment:
I updated the patch to fix the test_zipimport_support tests. All tests now
pass. The only changes were to Lib/test/test_zipimport_support.py.
I'll make a note about test_pyclbr and test_zipimport_support when I create the
issue to move the suppo
Changes by Chris Jerdonek :
--
nosy: +cjerdonek
___
Python tracker
<http://bugs.python.org/issue15806>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue15798>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Chris Rebert :
--
nosy: +cvrebert
___
Python tracker
<http://bugs.python.org/issue15806>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Jerdonek added the comment:
> In other words, (int(groups[0]), int(groups[1])) >= (int(previous_groups[0]),
> int(previous_groups[1])).
Why not use a single self.assertGreaterEqual() on the pairs (with an
appropriate change in the message)?
It currently hand-codes the lexi
Chris Jerdonek added the comment:
Attaching an updated patch that improves the organization of the new test cases
(test ordering, test names, and test comments, etc).
--
Added file: http://bugs.python.org/file27043/issue-15510-4.patch
___
Python
Chris Jerdonek added the comment:
This is a nit, but there should be two spaces after the period:
+each option in the section. Otherwise, return a list of tuples with
--
nosy: +cjerdonek
___
Python tracker
<http://bugs.python.org/issue15
Chris Jerdonek added the comment:
> If the double space is used elsewhere in the same document,
You can see one above the change in the patch file.
--
___
Python tracker
<http://bugs.python.org/issu
New submission from Chris Jerdonek:
>From d...@python.org:
"""
In a number of places we find documentation with optional leading arguments
where the meta-notation (square brackets) are in the wrong place. For
example, for range, the standard doc heading says
r
Chris Jerdonek added the comment:
Do we know that this is easy? It might be a Sphinx issue, in which case it
might not be as trivial as fixing typos.
I ask because the reST file looks correct (for range() for example):
.. function:: range([start,] stop[, step
Chris Jerdonek added the comment:
This is what my search for ",] " gave me (though the OP's print and random.seed
do not show up in this list):
library/curses.rst
380:.. function:: newwin([nlines, ncols,] begin_y, begin_x)
659:.. method:: window.addch([y, x,] ch[, attr]
Chris Jerdonek added the comment:
> It would be fine with me to use a double signature for these.
Just an FYI that more than two signatures would be needed for cases like this:
766:.. method:: window.chgat([y, x, ] [num,] attr)
--
___
Pyt
Chris Jerdonek added the comment:
> What's this even supposed to mean?
I started wondering the same thing after I posted. :)
I guess my point/question is: are there any cases where more than two
signatures would be needed to account for all of the possi
Chris Jerdonek added the comment:
The following commit from today is related to this issue: 815b88454e3e
"Remove trailing whitespace in order to silence warnings on HP-UX."
--
nosy: +cjerdonek
___
Python tracker
<http://bugs.python.
Chris Jerdonek added the comment:
> (The double signature might be easier to understand, but the original issue
> should probably be fixed in Sphinx, even if we decide to stop using this kind
> of signature.)
I filed an issue for this in the Sphinx tracker here:
https://bitb
Chris Jerdonek added the comment:
I will prepare a patch (multiple signatures, for the Python fix).
--
___
Python tracker
<http://bugs.python.org/issue15
Changes by Chris Rebert :
Added file: http://bugs.python.org/file27090/json.rst.patch
___
Python tracker
<http://bugs.python.org/issue14570>
___
___
Python-bugs-list m
Chris Rebert added the comment:
Attached updated patches.
--
Added file: http://bugs.python.org/file27091/json_init.patch
___
Python tracker
<http://bugs.python.org/issue14
Chris Jerdonek added the comment:
Something to be aware of that may or may not affect the patch I'm preparing:
One reason that Sphinx seems able to render some of the more complicated
function signatures is that it has logic to bail and print the parameter list
verbatim from the reST
Chris Jerdonek added the comment:
Here are a couple functions that may need a fix different from multiple
signatures:
print([object, ...], *, sep=' ', end='\n', file=sys.stdout, flush=False)
http://docs.python.org/dev/library/functions.html#print
class argparse.Argumen
Chris Jerdonek added the comment:
FWIW, I submitted a patch for the Sphinx issue I created:
https://bitbucket.org/birkenfeld/sphinx/issue/1001
Ironically, that patch was probably easier than this patch will be.
--
___
Python tracker
<h
New submission from Chris Jerdonek:
It seems like a couple error messages in the curses module need correcting:
PyErr_SetString(PyExc_TypeError, "insch requires 1 or 4 arguments");
http://hg.python.org/cpython/file/8ff2f4634ed8/Modules/_cursesmodule.c#l1322
PyErr_SetString(PyExc
Chris Jerdonek added the comment:
Attaching a proposed patch for the default branch. Also, here are several
comments and questions.
> I think/hope that all the APIs we have in the stdlib are sane enough to have
> no more than 2-3 signatures
I found this one in the curses module wit
Chris Jerdonek added the comment:
I just completed a patch to improve the documentation of these signatures
(along with many others) in issue 15831. The correction here is consistent
with my findings and revised documentation there.
As for the code, they are straightforward switch statements
Chris Jerdonek added the comment:
The situation is the same in 2.7 (and probably 3.2).
--
versions: +Python 2.7, Python 3.2
___
Python tracker
<http://bugs.python.org/issue15
New submission from Chris Jerdonek:
As of: changeset: 78843:8ff2f4634ed8
date:Sun Sep 02 16:37:09 2012 -0400
I am getting this failure:
$ ./python.exe -m test -v -u curses test_curses
== CPython 3.3.0rc1+ (default:8ff2f4634ed8, Sep 2 2012, 16:02:43) [GCC 4.2.1
Compatible Apple Clang
Changes by Chris Jerdonek :
--
title: curses test_curses test_unget_wch -> curses test_curses.test_unget_wch
crash
___
Python tracker
<http://bugs.python.org/issu
Chris Jerdonek added the comment:
Attaching a patch with tests and fix for the default branch.
--
keywords: +patch
Added file: http://bugs.python.org/file27105/issue-15831-1.patch
___
Python tracker
<http://bugs.python.org/issue15
Changes by Chris Jerdonek :
--
keywords: +needs review
stage: test needed -> patch review
___
Python tracker
<http://bugs.python.org/issue15852>
___
___
Python-
Chris Jerdonek added the comment:
Uploading correct file.
--
Added file: http://bugs.python.org/file27106/issue-15852-1.patch
___
Python tracker
<http://bugs.python.org/issue15
Changes by Chris Jerdonek :
Removed file: http://bugs.python.org/file27105/issue-15831-1.patch
___
Python tracker
<http://bugs.python.org/issue15852>
___
___
Python-bug
Chris Jerdonek added the comment:
Okay, thanks, Ned. Out of curiosity, why is the stack trace here different
from the one posted on that issue?
--
___
Python tracker
<http://bugs.python.org/issue15
Chris Jerdonek added the comment:
I added some comments on the latest patch on the review tool.
--
___
Python tracker
<http://bugs.python.org/issue15557>
___
___
Chris Jerdonek added the comment:
Thanks, David. I wasn't sure if you had seen the comments. They're mostly
stylistic, though, so it's not too big of a deal.
--
___
Python tracker
<http://bugs.pyt
Chris Jerdonek added the comment:
Fair enough. :) I may keep a couple of those changes in mind if I ever have a
chance to visit this module myself in the future.
--
___
Python tracker
<http://bugs.python.org/issue15
Changes by Chris Jerdonek :
--
nosy: +cjerdonek
___
Python tracker
<http://bugs.python.org/issue8911>
___
___
Python-bugs-list mailing list
Unsubscribe:
Chris Jerdonek added the comment:
> If you look at importlib.regrtest you will notice it has to muck with
> sys.argv in order to get certain tests skipped.
This is reminiscent of issue 15132 which I filed, but for
unittest.TestProgram() rather than regrtest.main().
> 6. Switch regrt
Chris Jerdonek added the comment:
Thanks, Andrew. Regarding your comment, it was a deliberate choice not to do
the additional check because I wanted each test to check only one thing. But I
am okay with adding the additional check.
Regarding the patch, should all of the methods now do
Chris Jerdonek added the comment:
Thanks a lot for the very thorough review, David. I should be able to update
the patch and respond to a couple of your points later today or tomorrow at the
latest.
--
___
Python tracker
<http://bugs.python.
Chris Jerdonek added the comment:
For future reference, here is the beginning of the e-mail thread on python-dev:
http://mail.python.org/pipermail/python-dev/2012-September/121584.html
We also need to know whether the Tru64 trick needs to be used in 2.7, since
this documentation issue also
1301 - 1400 of 2758 matches
Mail list logo
