[issue16353] add function to os module for getting path to default shell
New submission from Chris Jerdonek: This issue is to add a function to the os module for getting the path to the default shell (e.g. os.getdefaultshell()). In issue 16255, it was reported that on Android, the path to the default shell is /system/bin/sh rather than /bin/sh which it is on other Unix systems. Such a function in the os module would implement the necessary detection logic which could then be used, for example, by the subprocess module when invoking Popen() with shell=True (e.g. for issue 16255). -- components: Library (Lib) messages: 174093 nosy: chris.jerdonek priority: normal severity: normal status: open title: add function to os module for getting path to default shell type: enhancement versions: Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16353 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16255] subrocess.Popen needs /bin/sh but Android only has /system/bin/sh
Chris Jerdonek added the comment: I created issue 16353 for adding a function to the os module for getting the path to the default shell. The current issue 16255 could be addressed in 3.4 by calling such a function from the subprocess module. For earlier versions (since enhancements are not allowed), the current issue could perhaps be addressed by backporting the logic in such a function directly into the subprocess module (i.e. without adding a new function to the os module). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8040] documentation pages should link to other versions of the same page
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- nosy: +chris.jerdonek title: It would be nice if documentation pages linked to other versions of the same document - documentation pages should link to other versions of the same page ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8040 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16312] New command line option supported by unittest.main() for running initialization code before tests
Chris Jerdonek added the comment: Can you give an example of what you would like the API to look like (i.e. what you would like to be able to type from the command-line if, say, configuring logging as in the SO post)? I'm not sure I understand why a wrapper script isn't a good enough solution, as described in the accepted answer on SO. -- nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16312 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16331] Add a version switcher to python docs site
Chris Jerdonek added the comment: I recommend closing this as a duplicate. -- nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16331 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8040] documentation pages should link to other versions of the same page
Chris Jerdonek added the comment: Both patches (for 2.7 3.2) are almost the same as the original patch. Conflicts were trivial. For future reference, it's okay (and perhaps preferable) to upload a patch just for the default branch when no substantive changes are needed for other branches. This is especially true early in the review process when multiple iterations may be uploaded. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8040 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue15494] Move test/support.py into a test.support subpackage
Chris Jerdonek added the comment: Excessively large source files are a sign of bad software design. Do you [want to] weigh in the python-dev discussion about splitting unicodeobject.c? Nick did weigh in starting here: http://mail.python.org/pipermail/python-dev/2012-October/122391.html -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15494 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16319] optional comma inside function argument list triggers syntax error
Chris Jerdonek added the comment: I could be misinterpreting the documentation, but it looks to me like the examples given are behaving as documented: parameter_list ::= (defparameter ,)* ( * [parameter] (, defparameter)* [, ** parameter] | ** parameter | defparameter [,] ) (from http://docs.python.org/dev/reference/compound_stmts.html#function-definitions ) The relevant clause in the above looks to be the following, which can't give rise to a trailing comma: * [parameter] (, defparameter)* [, ** parameter] -- nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16319 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16319] optional comma inside function argument list triggers syntax error
Chris Jerdonek added the comment: Duplicate of issue 9232 (enhancement request). -- resolution: - duplicate stage: - committed/rejected status: open - closed superseder: - Allow trailing comma in any function argument list. ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16319 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue15494] Move test/support.py into a test.support subpackage
Chris Jerdonek added the comment: If you are planning to further split support.py, then I'm -1. Ezio, this issue is not to split up support.py. Nick already responded to you to this effect in the fourth comment above. Also, in the second comment to this issue, I said that this issue was to move support.py as is into support/__init__.py, and then to move script_helper.py as is into the package. Having several modules doesn't solve the problem of discoverability We already have more than one module. This issue helps discoverability by grouping them in one place so people know what support modules there are. In other words, this issue addresses the following problem you mentioned: Both these issues were probably caused by the fact that people didn't know about script_helper.py and ended up implementing their own versions of the functions either in the test files or in support.py. Something similar happened with requires_zlib (and possibly a few others)... So it is fair to ask what *specific* benefits such a restructuring may have. Martin, see the paragraph above. In addition, this change makes it easier to search whatever support modules we have. It also provides a place to put new support modules to avoid further exacerbating the discoverability problem stated above. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15494 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16163] Wrong name in Lib/pkgutil.py:iter_importers
Chris Jerdonek added the comment: Nick would be a better person to review the patch since IIRC he worked a bunch on that module (e.g. in the changeset you mentioned). But if I get a chance I will take a look. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16163 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16278] os.rename documentation slightly inaccurate
Chris Jerdonek added the comment: Before we discuss wording, it's not clear to me whether all possibilities have been checked. I count 2*3*2=12 possibilities to check (excluding src and dst being on different filesystems): src: file or directory dst: file, empty directory, or non-empty directory os: Unix or Windows Also, do we have tests for all of these scenarios before we document it? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16278 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue15132] Let unittest.TestProgram()'s defaultTest argument be a list
Chris Jerdonek added the comment: Thanks for the patch. -self.testNames = (self.defaultTest,) +if isinstance(self.defaultTest, str): +self.testNames = (self.defaultTest,) +else: +self.testNames = tuple(self.defaultTest) Is there any reason this is a tuple instead of a list? A list would be more flexible. In contrast, the _convert_names() function used in this line of code sets self.testNames to be a list of test names: self.testNames = _convert_names(args) (from http://hg.python.org/cpython/file/8576bf1c0302/Lib/unittest/main.py#l161 ) By the way, this can only go into Python 3.4 as it is an enhancement. -- versions: +Python 3.4 -Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15132 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue15916] change doctest DocTestSuite not to raise ValueError if no docstrings
Chris Jerdonek added the comment: Hmm. This patch still raises the has no docstrings ValueError if not tests is True. And because the patch changes the default DocTestFinder, it's possible that the patch changes behavior in non-error use cases. Why not just return an empty TestSuite if not tests is True? IIRC, this was the conclusion of the issue 14649 discussion. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15916 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16288] TextTestResult uses TestCase.__str__() which isn't customisable (vs id() or shortDescription())
Chris Jerdonek added the comment: testscenarios copies the tests, it doesn't call the constructor for the class; My suggestion on how to override __str__ (by assignment to __class__) doesn't require that the constructor be called. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16288 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16255] subrocess.Popen needs /bin/sh but Android only has /system/bin/sh
Chris Jerdonek added the comment: It occurs to me that logic for detecting the shell might make sense for being part of the os module, e.g. os.getdefaultshell(). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16288] TextTestResult uses TestCase.__str__() which isn't customisable (vs id() or shortDescription())
Chris Jerdonek added the comment: So the relevant code is this: def getDescription(self, test): ... 43 return str(test) ... What I'd like is to have this be something like: 44 return test.id() Or anther way this could be done would be to make TestCase.__str__ call self.id() Note that str(test) and test.id() don't return the same value. The former is the friendly name. It is probably intentional that TextTestResult uses the TestCase instance's friendly name in its getDescription() method as opposed to the id. TextTestRunner calls str(TestCase) directly, which makes it hard for testscenarios to rename the test cases as it parameterises them What about testscenarios subclassing the TestCase instances as it parametrizes them? class ScenariosTestCase(case.__class__): def __str__(self): return testscenarios test name... case.__class__ = ScenariosTestCase print(str(case)) From the documentation, it looks like testscenarios creates new test objects from the originals anyways. Alternatively (although I don't if testscenarios can control what test runner is being used), TextTestRunner could be subclassed with its own getDescription() method. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16288 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16282] The Python Tutorial the previous link needs updating
Chris Jerdonek added the comment: This issue does not seem limited to a particular page. For example, the Previous topic link for the Python Standard Library part: http://docs.python.org/library/index.html is 9. Full Grammar specification, which is the last section of the following part (the following part being the Language Reference). -- nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16282 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16282] The Python Tutorial the previous link needs updating
Chris Jerdonek added the comment: http://docs.python.org/tutorial/ the previous link goes to http://docs.python.org/whatsnew/2.0.html. This seems correct to me since the 'All What's new documents since 2.0' part of the documentation occurs immediately before the Tutorial part, and 2.0 is the last section in that 'All What's new' part. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16282 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16282] The Python Tutorial the previous link needs updating
Chris Jerdonek added the comment: Note that the 2.0 issue was previously reported in issue 4988 and issue 7683. The issue I described occurs because the parts on the home page are listed in an order different from their order in the table of contents: http://docs.python.org/ http://docs.python.org/contents.html So Previous topic respects the table of contents ordering and not the home page ordering. -- resolution: - invalid stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16282 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16288] TextTestResult uses TestCase.__str__() which isn't customisable (vs id() or shortDescription())
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16288 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16255] subrocess.Popen needs /bin/sh but Android only has /system/bin/sh
Chris Jerdonek added the comment: It seems simple enough for us to make the default configurable This seems good to do in any case (as opposed to having the string be hard-coded), and we can test it independently of Android. There is a recently added test in the 3.x branches that can be modified: test method test_executable_replaces_shell which checks that the executable argument replaces the default shell for non-Windows. I would suggest implementing Gregory's two suggested measures as two separate patches. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16266] Typo in tutorial
Chris Jerdonek added the comment: The last \n is needed for the reason Andrew mentioned, but it is true that the sentence could probably be rephrased to reduce the chance of confusion. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16266 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16266] Rephrase sentence in tutorial about universal newlines
Chris Jerdonek added the comment: I would suggest something like this: In text mode, the default when reading is to convert platform-specific line endings (\n on Unix, \r\n on Windows) to just \n. When writing in text mode, the default is to convert occurrences of \n back to platform-specific line endings. -- keywords: +easy title: Typo in tutorial - Rephrase sentence in tutorial about universal newlines versions: +Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16266 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16268] dir(closure) does not find __dir__
Chris Jerdonek added the comment: Here is a somewhat simpler way to reproduce (following the same definition of x): dir(x) ... TypeError: object does not provide __dir__ x.__dir__ built-in method __dir__ of cell object at 0x10c182950 dir(x) ['__class__', '__delattr__', '__dir__', '__doc__', '__eq__', '__format__', '__ge__', '__getattribute__', '__gt__', '__hash__', '__init__', '__le__', '__lt__', '__ne__', '__new__', '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__str__', '__subclasshook__', 'cell_contents'] The dir() documentation says, If the object has a method named __dir__(), this method will be called and must return the list of attributes. (from http://docs.python.org/py3k/library/functions.html#dir ) -- nosy: +chris.jerdonek title: dir(closure) claims that a closure has no __dir__, only to work later after manually invoking __dir__ from its type - dir(closure) does not find __dir__ versions: +Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16268 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16266] Rephrase sentence in tutorial about universal newlines
Chris Jerdonek added the comment: Committed. Thanks for the report! -- resolution: - fixed stage: - committed/rejected status: open - closed type: behavior - enhancement ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16266 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13538] Improve doc for str(bytesobject)
Chris Jerdonek added the comment: Attaching updated patch based on Ezio and Antoine's comments. Let me know if I'm not using the correct or preferred terminology around buffer objects and the buffer protocol. It doesn't seem like the section on the buffer protocol actually says what objects implementing the buffer protocol should be called. I gather indirectly from the docs that such objects are called buffer objects (as opposed to just buffers): http://docs.python.org/dev/c-api/buffer.html#bufferobjects -- stage: needs patch - patch review Added file: http://bugs.python.org/file27591/issue-13538-2-default.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13538 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13538] Improve doc for str(bytesobject)
Chris Jerdonek added the comment: Reattaching patch (a line was missing). -- Added file: http://bugs.python.org/file27592/issue-13538-3-default.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13538 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10836] urllib.request.urlretrieve calls URLError with 3 args
Chris Jerdonek added the comment: This should be reported in another issue though. Has this been done? -- nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10836 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16250] URLError invoked with reason as filename
New submission from Chris Jerdonek: This issue is to fix invocations of URLError so that reasons are not passed in for the filename argument of the URLError constructor. Ezio found and described this issue when commenting on issue 10836: http://bugs.python.org/issue10836#msg172988 In more detail: URLError accepts reason and filename arguments: http://hg.python.org/cpython/file/74b95194ba86/Lib/urllib/error.py#l23 However, in several places in Lib/urllib, URLError is invoked with a reason passed in for the filename. For example: raise URLError('local file error', 'not on local host') This issue could be addressed for this example by changing the above to: raise URLError('local file error: not on local host') This dovetails with issue #16247, which is to start passing the actual filename in for some calls to URLError (and perhaps to adjust URLError's __str__, etc). -- components: Library (Lib) keywords: easy messages: 173058 nosy: chris.jerdonek, ezio.melotti priority: normal severity: normal status: open title: URLError invoked with reason as filename type: enhancement versions: Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16250 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10836] urllib.request.urlretrieve calls URLError with 3 args
Chris Jerdonek added the comment: For tracking purposes, I created issue 16250 for what you observed above, which as you said is related to but not the same as Nick's issue 16247. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10836 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16250] URLError invoked with reason as filename
Chris Jerdonek added the comment: raise URLError('local file error: not on local host') I should also point out that URLError invocations with formats like the above already do occur in Lib/urllib/request.py. For example: raise URLError('ftp error: no host given') exc = URLError('ftp error: %s' % msg) raise URLError(http protocol error: bad status line) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16250 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16252] bytes and bytearray methods are undocumented
Chris Jerdonek added the comment: I couldn't find any doc for bytes.join, etc. One of the contributing factors to this is that bytes doesn't have its own class entry in the same way that, for example, list, dict, and memoryview have. This creates a stumbling block for readers because they have to know to click on the Bytes and Bytearray Operations section as opposed to simply clicking on the type name as it is done for several other types. With a class entry, locating where the methods are documented would be quicker and more intuitive. -- nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16252 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16250] URLError invoked with reason as filename
Chris Jerdonek added the comment: For future reference, this affects 3.2 onwards. It probably makes sense to address this in the same branches in which we plan to address issue 16247 (currently set to 3.3 onwards). -- stage: - needs patch versions: +Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16250 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16255] subrocess.Popen needs /bin/sh but Android only has /system/bin/sh
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- nosy: +asvetlov, chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13538] Improve doc for str(bytesobject)
Chris Jerdonek added the comment: I would just say that str(bytesobj, encoding, errors) is equivalent to bytesobj.decode(encoding, errors) (assuming it really is). Good suggestion. And yes, code is shared in the following way: http://hg.python.org/cpython/file/d3c7ebdc71bb/Objects/bytesobject.c#l2306 One thing that would need to be addressed in the str() version is if bytesobj is a PEP 3118 character buffer, after which it falls back to bytesobj.decode(encoding, errors). I will update the patch so people can see how it looks. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13538 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16227] Add link to 3.3 version in unicode howto
Chris Jerdonek added the comment: Committed without changing the first sentence, as Éric preferred. Thanks for the report, Mark! -- resolution: - fixed stage: needs patch - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16227 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] add to looping techniques tutorial a note about modifying sequence
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- title: list.remove in for loop - add to looping techniques tutorial a note about modifying sequence type: - enhancement versions: +Python 3.2, Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] add to looping techniques tutorial a note about modifying sequence
Chris Jerdonek added the comment: Thanks, Raymond. I will be doing that later today. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] add to looping techniques tutorial a note about modifying sequence
Chris Jerdonek added the comment: Committed. Thanks for reporting the suggestion, Ian. -- resolution: - fixed stage: patch review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] list.remove in for loop
Chris Jerdonek added the comment: I have worked with languages where you are explicitly warned that you must not mess with the loop variable There is a warning in this part of the tutorial: It is not safe to modify the sequence being iterated over in the loop... (from http://docs.python.org/dev/tutorial/controlflow.html#for-statements ) But it may be good to add a note to the section you reference as well. -- assignee: - docs@python components: +Documentation nosy: +chris.jerdonek, docs@python ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] list.remove in for loop
Chris Jerdonek added the comment: Attached is a simple way of addressing this (essentially copying the verbiage and example from the other page). If we want, we could make the sample code different so that the reader doesn't see the same thing twice. -- keywords: +patch stage: - patch review Added file: http://bugs.python.org/file27562/issue-16225-1-default.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16227] Change unicode howtos as misleading
Chris Jerdonek added the comment: I wonder why the note must be there at all. None of the 2.x docs have been updated to cover the 3.x versions of Python. Otherwise, every 2.x page could use such a disclaimer. Here is where the note was added: http://hg.python.org/cpython/rev/bdef454f7212 -- nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16227 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] list.remove in for loop
Chris Jerdonek added the comment: It is safe to modify a sequence during iteration if it's size not increased. What do you mean by safe? The example given by the original commenter does not increase the size either. I believe safe is meant in the sense of avoiding possibly unexpected behavior. I can clarify the language if you feel it is ambiguous. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16227] Change unicode howtos as misleading
Chris Jerdonek added the comment: I don't think there is a need to update the 3.1.x version of the documentation in that respect. versions: +Python 3.3, Python 3.4 -Python 3.1 This issue was just about the 2.7 version. 3.x doesn't contain the language mentioned by the original commenter. If 2.7 should not be changed, this issue should be closed. Or are you suggesting other updates to the 3.x docs? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16227 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16227] Change unicode howtos as misleading
Chris Jerdonek added the comment: 3.x doesn't contain the language mentioned by the original commenter. 3.2 onwards I mean. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16227 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16222] server-side clone not found by devguide's search box
Chris Jerdonek added the comment: I filed this in case it signals some underlying issue with search that could use improving (and that may affect other search terms). I can change the title to remove the focus on those two terms. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16222 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16222] some terms not found by devguide's search box
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- title: server-side clone not found by devguide's search box - some terms not found by devguide's search box ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16222 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] list.remove in for loop
Chris Jerdonek added the comment: But in general the advice should be: if you want to insert or remove elements during iteration, iterate over a copy. I would expand this to cover changing the list in any way. I think the point being made is that iteration doesn't implicitly make a copy. There are cases where modifying the list in place can also yield unexpected results. For example (naive list reversal): words = ['cat', 'window', 'defenestrate'] for i, word in enumerate(words): ... words[-i-1] = word ... words ['cat', 'window', 'cat'] -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16227] Change unicode howtos as misleading
Chris Jerdonek added the comment: That proposal sounds fine to me. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16227 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16227] Add link to 3.3 version in unicode howto
Chris Jerdonek added the comment: Attaching proposed patch. -- keywords: +patch Added file: http://bugs.python.org/file27571/issue-16227-1-27.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16227 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] list.remove in for loop
Chris Jerdonek added the comment: Attaching revised patch. -- Added file: http://bugs.python.org/file27573/issue-16225-2-default.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16227] Add link to 3.3 version in unicode howto
Chris Jerdonek added the comment: I would not rephrase the first two lines. Why not? The current language focuses on the negative (explaining problems) while we prefer an affirmative tone: http://docs.python.org/devguide/documenting.html#affirmative-tone -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16227 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] list.remove in for loop
Chris Jerdonek added the comment: Reattaching. I duplicated a variable definition that was defined previously. -- Added file: http://bugs.python.org/file27575/issue-16225-3-default.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] list.remove in for loop
Chris Jerdonek added the comment: I mean it does not lead to crash, hang, etc. I agree. I removed the word safe in the patch I attached to reduce ambiguity. Regarding unexpected behavior, remember that the tutorial is for beginners/newcomers. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16225] list.remove in for loop
Chris Jerdonek added the comment: It will be nice somewhere in deep clarify for experts what happens with list iterator if the list changed. There is a note somewhat to this effect here: http://docs.python.org/dev/reference/compound_stmts.html#the-for-statement Note There is a subtlety when the sequence is being modified by the loop -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16225 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16206] dict() docs should display multiple signatures
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16206 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13538] Improve doc for str(bytesobject)
Chris Jerdonek added the comment: Attaching a proposed patch along the lines suggested by Éric. -- keywords: +patch versions: +Python 3.4 Added file: http://bugs.python.org/file27556/issue-13538-1-default.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13538 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue15936] Add link from os.urandom to random.SystemRandom
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- nosy: +christian.heimes ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15936 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16222] server-side clone not found by devguide's search box
New submission from Chris Jerdonek: Neither searching for server-side nor clone in the devguide Quick Search box pulls up the mention of server-side clones: http://docs.python.org/devguide/committing.html#long-term-development-of-features -- components: Devguide messages: 172849 nosy: chris.jerdonek, eric.araujo, ezio.melotti, ncoghlan priority: normal severity: normal status: open title: server-side clone not found by devguide's search box type: enhancement ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16222 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16206] dict() docs should display multiple signatures
New submission from Chris Jerdonek: The documentation for dict() (for both the built-in function and the class constructor) should display multiple signatures as do dict.__doc__ and the documentation for, say, str(). Currently, the dict() documentation just has one signature: dict([arg]) Something more in line with the docstring would be better, for example: dict(mapping={}) dict(iterable) dict(**kwargs) -- assignee: docs@python components: Documentation keywords: easy messages: 172718 nosy: chris.jerdonek, docs@python, ezio.melotti priority: normal severity: normal status: open title: dict() docs should display multiple signatures versions: Python 2.7, Python 3.2, Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16206 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16207] add class name to method signatures in stdtypes docs
New submission from Chris Jerdonek: The method signatures for some of the classes documented in library/stdtypes lack the class name. For example, the list, dict, set, and memoryview methods lack the class name. Adding the class name will help to distinguish methods from functions. -- assignee: docs@python components: Documentation messages: 172720 nosy: chris.jerdonek, docs@python, ezio.melotti priority: normal severity: normal stage: needs patch status: open title: add class name to method signatures in stdtypes docs type: enhancement versions: Python 2.7, Python 3.2, Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16207 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16207] add class name to method signatures in stdtypes docs
Chris Jerdonek added the comment: I think I would be more okay with this if it weren't for the fact that some functions also appear indented under their class entries (making them look like methods). Right now, there's no visual distinction between the functions and the methods. Prefixing the methods with the class name or moving the functions above the class definition would address this. -- status: pending - open ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16207 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16207] add class name to method signatures in stdtypes docs
Chris Jerdonek added the comment: So it looks like the class name gets stripped when rendering if you prefix the method name with the class name and the methods are nested beneath the class. So it seems like the options are to either (1) move the non-methods outside the class (e.g. before the class), or (2) clearly divide the methods and non-methods into two groups (e.g. These are the operations... and These are the methods...). I lean towards the latter. And it would be a smaller change. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16207 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16209] add a class str entry to the docs
New submission from Chris Jerdonek: This issue is to add a str class entry to the built-in types page like we have for dict, set, memoryview, etc: http://docs.python.org/dev/library/stdtypes.html This will let us reference ``str`` the class separately from ``str()`` the built-in function. It will also let us remove the str. prefix from all of the string method descriptions. Thirdly, it will make the str documentation match what is being done for the types previous to it in the documentation, including the range, tuple, and list classes. Lastly, it will make referencing string-related sections in Sphinx a bit more intuitive. Currently, :class:`str` points to the built-in function :func:`str`. -- assignee: docs@python components: Documentation messages: 172733 nosy: chris.jerdonek, docs@python, ezio.melotti priority: normal severity: normal status: open title: add a class str entry to the docs type: enhancement versions: Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16209 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16207] distinguish methods from non-methods in classes in the stdtypes docs
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- title: add class name to method signatures in stdtypes docs - distinguish methods from non-methods in classes in the stdtypes docs ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16207 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16210] combine the two type() definitions in built-in function docs
New submission from Chris Jerdonek: [Found by Ezio] The built-in function documentation has what looks like two separate definitions of the built-in function type(): http://docs.python.org/dev/library/functions.html#type These two definitions should be combined into one with a multi-line signature like we're doing for the other built-in functions. -- assignee: docs@python components: Documentation messages: 172734 nosy: chris.jerdonek, docs@python, ezio.melotti priority: normal severity: normal status: open title: combine the two type() definitions in built-in function docs versions: Python 2.7, Python 3.2, Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16210 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16205] update :class:`str` references to link to the str type section
Chris Jerdonek added the comment: If :class:`str` and :func:`str` point respectively to stdtypes.rst and functions.rst once a class directive is added to stdtypes.rst, then we can do that. I agree. I would suggest starting small by adding a stub class entry above the string methods. The class constructor could link to the built-in function str() for its definition. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16205 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16206] dict() docs should display multiple signatures
Chris Jerdonek added the comment: Attaching proposed patch. -- keywords: +patch Added file: http://bugs.python.org/file27546/issue-16206-1-default.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16206 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16209] add a class str entry to the docs
Chris Jerdonek added the comment: Let me start over with what I'm suggesting, because the idea wasn't fully developed when I first filed this issue. I'm thinking that we should create a class entry for str that contains the current string methods nested beneath it -- a bit like issue 12901. :) In conjunction with that, we should move the documentation of the str() function to the constructor for that class entry. All str links throughout the documentation would go to the class entry, and the built-in function str() would be a stub entry that also links to the class entry. That's how we do it for dict, for example: http://docs.python.org/dev/library/functions.html#func-dict One reason is practical. By doing the above, clicking str anywhere in the docs will take you to the documentation for both the function str() and the string methods. When I was first learning Python, it was very common for me to look up the String Methods section. But looking them up never seemed as convenient as it should be because they were in a different location. I always had to remember they were an extra click or two away in a separate String Methods section. In 3.3 we made a bit of progress on this by giving the string type its own section: http://docs.python.org/dev/library/stdtypes.html#text-sequence-type-str But we could go further by doing what we do for list, tuple, set, memoryview, range, etc. and put the documentation of the str() function and its methods next to each other in the same location. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16209 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16192] ctypes - documentation example
Chris Jerdonek added the comment: Perhaps if some machines expect otherwise that should be documented, otherwise might we change them. The very beginning of the ctypes documentation has documentation to that effect: Note: Some code samples reference the ctypes c_int type. This type is an alias for the c_long type on 32-bit systems. So, you should not be confused if c_long is printed if you would expect c_int — they are actually the same type. http://docs.python.org/dev/library/ctypes.html#ctypes-tutorial Does that address your suggestion? -- nosy: +chris.jerdonek status: open - pending ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16192 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16192] ctypes - documentation example
Chris Jerdonek added the comment: I don't know off-hand. But maybe an additional note or code comment at the first instance in a code example would help. Switching the examples to c_long might serve only to shift potential confusion from the examples to someone's personal machine. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16192 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13440] Explain the status quo wins a stalemate principle in the devguide
Chris Jerdonek added the comment: I'll say it again, the devguide should be short enough to be practical for someone learning to contribute. Better organization could help here. I could see the devguide being a combination of (1) a brief document meant to be read cover to cover that captures the minimum one needs to know (as well as describing what else is available in the guide), and (2) more detailed info that someone could read on an as-needed basis or that a core developer could link to if assisting someone on a certain point (like the point covered by this issue). Currently, the devguide is harder to soak in because there is no boundary between (1) and (2). A recent example is that I didn't know about the custom repo option until Antoine mentioned it in a comment -- even after having looked at many of the sections. On the plus side, he was able to link to it with the current organization. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13440 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13440] Explain the status quo wins a stalemate principle in the devguide
Chris Jerdonek added the comment: Would this be the appropriate place for the links to the two essays: Personally, I would start out with a question in the FAQ. It could be called something like, When is a change to Python justified? and go after or in the same section as the question, Where should I suggest new features and language changes? Then it could be referenced from other sections as needed (e.g. from Changing the Python Language, Reviewing Patches, etc). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13440 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue15939] make *.rst files in Doc/ parseable by doctest
Chris Jerdonek added the comment: Either there's another way to run those tests on selected platforms that can be used instead A way to do it could be to register new doctest directives that skip the marked test based on the value of sys.platform. If we remove the directives, we lose that information and won't be able to do it anymore. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15939 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16193] display full e-mail name in hg.python.org annotate pages
Chris Jerdonek added the comment: This is the suggested change in the bug report I linked to above (providing we are using 2.3 or later): If you'd like the latter in hgweb, you can get it by editing '|user' [to '|emailuser'] in the templates. (from http://bz.selenic.com/show_bug.cgi?id=2276#c7 ) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16193 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue15939] make *.rst files in Doc/ parseable by doctest
Chris Jerdonek added the comment: FTR there were a few actual errors that I fixed (mostly about bytes vs string). Great! Note that for the others, I was just trying to make the files parseable rather than passing. Currently, doctest aborts out on the files I fixed without even attempting to run the examples. That can be done for 3.4+ only though. Well, yes and no. If there is a helper script to check documentation, it could be run on any branch regardless of whether the script is in source control. For example, I was planning to write such a script (from the regrtest patch I put together before) to help me when checking people's patches to code snippets in the documentation. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15939 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16199] 2to3 very minor doc change
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- assignee: docs@python - chris.jerdonek nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16199 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16199] 2to3 very minor doc change
Chris Jerdonek added the comment: Fixed. Thanks for the report, Mark! -- resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16199 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16205] update :class:`str` references to link to the str type section
New submission from Chris Jerdonek: 3.3 added a dedicated section to the docs for the str type. However, references to :class:`str` still link to the documentation of the built-in function str(). This issue is to update references to the str class to point to the new section on the str type. We could use :ref:`str textseq`, but it would be nice to have a solution that makes str have the code-style font instead of italics. -- assignee: docs@python components: Documentation messages: 172713 nosy: chris.jerdonek, docs@python, ezio.melotti, ncoghlan priority: normal severity: normal status: open title: update :class:`str` references to link to the str type section type: enhancement versions: Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16205 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16205] update :class:`str` references to link to the str type section
Chris Jerdonek added the comment: I'm not sure if Sphinx would allow it, but it would be nice if references to the function str() could link to the built-in function documentation, and references to the class str could link to the section on the str type. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16205 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13538] Improve doc for str(bytesobject)
Chris Jerdonek added the comment: This may have been addressed to some extent by issue 14783: http://hg.python.org/cpython/rev/3773c98d9da8 -- nosy: +chris.jerdonek ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13538 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16205] update :class:`str` references to link to the str type section
Chris Jerdonek added the comment: The documentation seems to take a similar approach with dict: http://docs.python.org/dev/library/stdtypes.html#dict http://docs.python.org/dev/library/functions.html#func-dict -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16205 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16114] incorrect path in subprocess.Popen() FileNotFoundError message
Chris Jerdonek added the comment: replacing repr(args[0]) with repr(executable) in the identified python should be sufficient for this bug as originally reported. Yes, almost. The executable variable is a bytes object so it needs to be fsdecoded first. I have a test and fix for that. I'll take care of both. Let me at least upload what I already prepared. You can add to it or modify it as you see fit. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16114 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16114] incorrect path in subprocess.Popen() FileNotFoundError message
Chris Jerdonek added the comment: I made some comments on the changes as committed here: http://mail.python.org/pipermail/python-dev/2012-October/122125.html It would be cleaner to use the self.assertRaises() pattern here and also probably better to share code across the three test methods which are nearly identical to one another (there is a fourth scenario I would also add of shell=True). I would also check for FileNotFoundError instead of OSError in the 3.3 and later versions. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16114 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16185] include path in subprocess.Popen() file not found error messages on Windows
New submission from Chris Jerdonek: Currently, when subprocess.Popen() raises a FileNotFoundError on POSIX systems, the error message helpfully includes the path not found. But it does not do so on Windows. This issue is to include the path also in the messages of the corresponding errors on Windows. See also the related issue 16114. -- components: Library (Lib) messages: 172569 nosy: asvetlov, chris.jerdonek, gregory.p.smith priority: normal severity: normal status: open title: include path in subprocess.Popen() file not found error messages on Windows type: enhancement versions: Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16185 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16114] incorrect path in subprocess.Popen() FileNotFoundError message
Chris Jerdonek added the comment: Maybe better to fix Windows behavior for unifying FileNotFoundError? I created issue 16185 to include the path in the messages of the corresponding errors on Windows. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16114 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12947] doctest directive examples in library/doctest.html lack the flags
Chris Jerdonek added the comment: Yes, the workaround should be applied in those cases as well. I added the workaround to the last remaining example. Nick was right before in that three of the four examples I mentioned above were in the part about option flags rather than in the part about directives. I made those two parts into their own sections for added clarity. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12947 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16187] incoming.irker hook raised an exception: [Errno 111] Connection refused
New submission from Chris Jerdonek: I got the following irker-related errors when committing about 20 minutes ago: remote: added 3 changesets with 2 changes to 1 files remote: buildbot: change(s) sent successfully remote: sent email to roundup at rep...@bugs.python.org remote: notified python-check...@python.org of incoming changeset 00555659253d remote: error: incoming.irker hook raised an exception: [Errno 111] Connection refused remote: notified python-check...@python.org of incoming changeset 467c9f663b77 remote: error: incoming.irker hook raised an exception: [Errno 111] Connection refused remote: notified python-check...@python.org of incoming changeset 15bfa778ec21 remote: error: incoming.irker hook raised an exception: [Errno 111] Connection refused -- components: None messages: 172577 nosy: chris.jerdonek, ezio.melotti, georg.brandl priority: normal severity: normal status: open title: incoming.irker hook raised an exception: [Errno 111] Connection refused type: behavior ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16187 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12947] doctest directive examples in library/doctest.html lack the flags
Chris Jerdonek added the comment: Thanks, Georg. Can this new directive be applied to individual code blocks though (or be modified to do so)? After a closer review (as mentioned in my comment above), it seems that the doctest directives should only be displayed in the code snippets that are illustrating the use of doctest directives (as opposed to, say, the snippets that are illustrating doctest options without using directives). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12947 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12947] doctest directive examples in library/doctest.html lack the flags
Chris Jerdonek added the comment: Like these that discuss the behavior of the doctest.IGNORE_EXCEPTION_DETAIL option (but before doctest directives have been introduced): http://hg.python.org/cpython/file/0d6910bef979/Doc/library/doctest.rst#l543 The key is that you can have options take effect without using directives, so it would be good if people can see examples like that. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12947 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12947] doctest directive examples in library/doctest.html lack the flags
Chris Jerdonek added the comment: Georg suggested removing the doctest directives from the examples I mentioned. As currently written, they don't help them pass if doctest were run on the .rst file. That sounds fine to me for now. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12947 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16115] test the executable arg to Popen() and improve related docs
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16115 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16187] incoming.irker hook raised an exception: [Errno 111] Connection refused
Chris Jerdonek added the comment: It's working now, thanks. -- status: pending - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16187 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12947] doctest directive examples in library/doctest.html lack the flags
Chris Jerdonek added the comment: This seems to look good now. Can this be closed, or is there something else that needed to be done as well? -- resolution: - fixed stage: needs patch - commit review status: open - pending ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12947 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16193] display full e-mail name in hg.python.org annotate pages
New submission from Chris Jerdonek: This issue is to configure hg.python.org to display the full e-mail name on the annotate pages. Currently, instead of displaying everything before the @ symbol, the pages display only the portion of e-mail addresses up to the first . or @, for example tim@19837. Information on how to configure hgweb this way is here: http://bz.selenic.com/show_bug.cgi?id=2276 -- messages: 172621 nosy: chris.jerdonek, ezio.melotti, georg.brandl, pitrou priority: normal severity: normal status: open title: display full e-mail name in hg.python.org annotate pages type: enhancement ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16193 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16115] test that executable arg to Popen() takes precedence over args
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- resolution: - fixed stage: patch review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16115 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16161] broken link to Visual Studio 2008 in devguide
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- resolution: - fixed stage: needs patch - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16161 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16114] incorrect path in subprocess.Popen() FileNotFoundError message
Chris Jerdonek added the comment: BTW, the patch fails for 3.2 and 3.3 but works for 3.4 By patch do you mean test? And by works, do you mean fails or succeeds? :) I haven't prepared a patch yet, but I just started working on it. On my machine, I found that the test fails as is on 3.3 and 3.4, but on 3.2 it fails for a different reason: the FileNotFoundError needs to be switched to OSError (which was to be expected). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16114 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue15922] make howto/urllib2.rst doctests pass
Chris Jerdonek added the comment: Thanks, Senthil! -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15922 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com