[issue10639] reindent.py should not convert newlines

Scott Dial added the comment: I haven't seen anyone use a side-effect-less statement (a string) as a comment before, but I doubt that is an approved style for the CPython codebase. Please change the string preceeding the spec_line definition into a proper comment. -- nosy: +scott.dial

[issue12680] cPickle.loads is not thread safe due to non-thread-safe imports

New submission from Sagiv Malihi : When trying to cPickle.loads() from several threads at once, there is a race condition when threads try to import modules. An example will explain it best: suppose I have module foo.py which takes some time to load: import time class A(object): def __sets

[issue12680] cPickle.loads is not thread safe due to non-thread-safe imports

Sagiv Malihi added the comment: OK, digging deeper reveals that there are actually two bugs here, one is conceptual in the python importing mechanism, and the other is technical in cPickle. The first bug: PyImport_ExecCodeModuleEx adds the module to sys.modules *before* actually executing t

[issue1813] Codec lookup failing under turkish locale

Stefan Krah added the comment: Unrelated to the Fedora issue: The test is currently skipped on the FreeBSD bot, but completes successfully with: diff -r 0b52b6f1bfab Lib/test/test_locale.py --- a/Lib/test/test_locale.py Tue Aug 02 10:16:45 2011 +0200 +++ b/Lib/test/test_locale.py Tue Aug 02

[issue1813] Codec lookup failing under turkish locale

Stefan Krah added the comment: As I wrote on python-dev, this test also fails on Debian lenny, which has the same setlocale() bug as Fedora. So, indeed the test should be skipped on a multitude of platforms. -- ___ Python tracker

[issue12676] Bug in http.client

Roundup Robot added the comment: New changeset 1013c9fbd83c by Senthil Kumaran in branch '3.2': Fix closes Issue12676 - Invalid identifier used in TypeError message in http.client. http://hg.python.org/cpython/rev/1013c9fbd83c -- nosy: +python-dev resolution: -> fixed stage: needs pat

[issue12676] Bug in http.client

Roundup Robot added the comment: New changeset c099ba0a278e by Senthil Kumaran in branch 'default': Fix closes Issue12676 - Invalid identifier used in TypeError message in http.client. http://hg.python.org/cpython/rev/c099ba0a278e -- ___ Python trac

[issue12676] Bug in http.client

Senthil Kumaran added the comment: Thanks for the bug report, Popa Claudiu and Patch Santoso Wijaya. Ouch, pretty ugly bug - shows the code lacked test coverage. -- ___ Python tracker

[issue12680] cPickle.loads is not thread safe due to non-thread-safe imports

Changes by R. David Murray : -- nosy: +brett.cannon, ncoghlan ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue12183] Document behaviour of shutil.copy2 and copystat with symlinks

Roundup Robot added the comment: New changeset 209ad8920b03 by Senthil Kumaran in branch '2.7': Fix closes Issue12183 - Explain the Symlink copy behavior in shutil.copytree. Patch by Petri Lehtinen. http://hg.python.org/cpython/rev/209ad8920b03 -- nosy: +python-dev resolution: -> fixe

[issue12183] Document behaviour of shutil.copy2 and copystat with symlinks

Roundup Robot added the comment: New changeset 22730c87 by Senthil Kumaran in branch '3.2': Fix closes Issue12183 - Explain the Symlink copy behavior in shutil.copytree. Patch by Petri Lehtinen. http://hg.python.org/cpython/rev/22730c87 -- _

[issue12183] Document behaviour of shutil.copy2 and copystat with symlinks

Senthil Kumaran added the comment: Petri, Thanks for the patch. It would also be helpful to track - "shutil.copytree() use lutimes in Python 3.3 to copy symlink metadata if symlinks=True". You can raise a feature request if it is not already raised. Thanks!. --

[issue10639] reindent.py should not convert newlines

R. David Murray added the comment: Well, this is actually blessed by http://www.python.org/dev/peps/pep-0257/, but if that convention were actually followed the docstring would go *after* the assignment. But I agree that it is rarely used, and as far as I know is not used in the stdlib at al

[issue1813] Codec lookup failing under turkish locale

R. David Murray added the comment: On Tue, 02 Aug 2011 12:12:37 +0200, Stefan Krah wrote: > I suspect many buildbots are green because they don't have tr_TR and > tr_TR.iso8859-9 installed. This is true for my Gentoo buildbots. Once we've figured out the best way to handle this, I'll fix that

[issue12681] unittest expectedFailure could take a message argument like skip does

New submission from R. David Murray : I think that it would be good for expectedFailure to take a message argument like skip does. My thought is that it would be printed both when it is triggered (the test fails as expected) so that one case from the verbose output why the test is expected to

[issue1813] Codec lookup failing under turkish locale

Stefan Krah added the comment: [Re-opening to fix the skips] Yes, the test works on: Ubuntu Lucid (libc-2.11.1), OpenSUSE (libc-2.11.1), FreeBSD-8.2 Failure: Fedora 14 (libc-2.13), Debian lenny (libc-2.7), Gentoo (libc-2.13-r2) So perhaps this test should be marked as expected failure

[issue1813] Codec lookup failing under turkish locale

Antoine Pitrou added the comment: > As someone pointed out on python-dev, if this isn't fixable then it > should be an expected failure, not a skip. The Python bug is fixed, the problem is apparently some libcs have the same bug as we did... > One question is, is there any platform on which th

[issue10639] reindent.py should not convert newlines

Roundup Robot added the comment: New changeset dc96af0e7f60 by Jason R. Coombs in branch 'default': Corrected attribute docstring per pep-257 (reference #10639) http://hg.python.org/cpython/rev/dc96af0e7f60 -- ___ Python tracker

[issue8639] Allow callable objects in inspect.getfullargspec

Changes by Maxim Bublis : Removed file: http://bugs.python.org/file22825/inspect.patch ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue12183] Document behaviour of shutil.copy2 and copystat with symlinks

Roundup Robot added the comment: New changeset 746dc0a2398e by Ezio Melotti in branch 'default': #12183: merge with 3.2. http://hg.python.org/cpython/rev/746dc0a2398e -- ___ Python tracker

[issue12670] Fix struct code after forward declaration on ctypes doc

Roundup Robot added the comment: New changeset 2aa8dd4df650 by Sandro Tosi in branch '2.7': #12670: Fix struct code after forward declaration on ctypes doc http://hg.python.org/cpython/rev/2aa8dd4df650 New changeset 25dd1d3f4b88 by Sandro Tosi in branch '3.2': #12670: Fix struct code after forw

[issue12183] Document behaviour of shutil.copy2 and copystat with symlinks

Senthil Kumaran added the comment: Hi Ezio, It was intentional that I did not merge it to default. We want shutil.copystat to use lutimes which is available in 3.3 and in which case, we will have to remove this once that change is done. -- ___ Pytho

[issue12183] Document behaviour of shutil.copy2 and copystat with symlinks

Ezio Melotti added the comment: I applied it because the patch is valid on 3.3 too until we actually change copystat to use lutimes. If/when shutil is changed the doc can be updated accordingly. -- nosy: +ezio.melotti ___ Python tracker

[issue12670] Fix struct code after forward declaration on ctypes doc

Changes by Sandro Tosi : -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker ___

[issue12682] Meaning of 'accepted' resolution as documented in devguide

New submission from R. David Murray : The devguide documents the 'accepted' resolution as follows: Submitted patch was applied, still needs verifying (for example by watching the buildbots) that everything went fine. At that point the resolution should be set to fixed and the stat

[issue12682] Meaning of 'accepted' resolution as documented in devguide

Changes by Ezio Melotti : -- components: +Devguide -Documentation nosy: +ezio.melotti resolution: -> accepted stage: -> needs patch ___ Python tracker ___ _

[issue9561] distutils: set encoding to utf-8 for input and output files

Michał Górny added the comment: Ping. What's the progress on this? Will this ever be fixed? -- ___ Python tracker ___ ___ Python-bugs-

[issue12646] zlib.Decompress.decompress/flush do not raise any exceptions when given truncated input streams

Antoine Pitrou added the comment: I'm not a zlib specialist, but I think what this means is that the stream is not finished, but it's valid anyway. For example, you get the same behaviour by doing: c = zlib.compressobj() s = c.compress(b'This is just a test string.') s += c.flush(zlib.Z_FULL_

[issue12682] Meaning of 'accepted' resolution as documented in devguide

Changes by Daniel Stutzbach : -- nosy: +stutzbach ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.p

[issue12646] zlib.Decompress.decompress/flush do not raise any exceptions when given truncated input streams

Oleg Oshmyan added the comment: I like the new patch, but shouldn’t the default be to behave the same way zlib.decompress() behaves, i. e. raise? (Or perhaps zlib.decompress() should be modified not to raise instead. I’m just aiming at consistency.) Of course this will break code that relies

[issue12682] Meaning of 'accepted' resolution as documented in devguide

Ross Lagerwall added the comment: > Formally I believe that the 'accepted' resolution is for committed feature > requests That's what I also thought originally. Perhaps this should be discussed on Python-Dev to see what people think it should mean? -- nosy: +rosslagerwall _

[issue12665] Dictionary view example has error in set ops

Roundup Robot added the comment: New changeset a70cdec027e7 by Sandro Tosi in branch '3.2': #12665: Dictionary view example has error in set operation http://hg.python.org/cpython/rev/a70cdec027e7 New changeset b2dc821058fe by Sandro Tosi in branch 'default': #12665: merge with 3.2 http://hg.py

[issue12665] Dictionary view example has error in set ops

Changes by Sandro Tosi : -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker ___

[issue12681] unittest expectedFailure could take a message argument like skip does

Changes by Santoso Wijaya : -- nosy: +santa4nt ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyth

[issue11933] newer() function in dep_util.py mixes up new vs. old files due stat.st_mtime vs stat

Roundup Robot added the comment: New changeset aebe3243bb2c by Éric Araujo in branch '2.7': Fix incorrect mtime comparison in distutils (#11933). http://hg.python.org/cpython/rev/aebe3243bb2c -- nosy: +python-dev ___ Python tracker

[issue12679] ThreadError is not in threading.__all__

Roundup Robot added the comment: New changeset bbeda42ea6a8 by Benjamin Peterson in branch 'default': add ThreadError to threading.__all__ (closes #12679) http://hg.python.org/cpython/rev/bbeda42ea6a8 -- nosy: +python-dev resolution: -> fixed stage: -> committed/rejected status: open

[issue12678] test_packaging and test_distutils failures under Windows

Éric Araujo added the comment: Here’s the distutils error: ERROR: test_manual_manifest (distutils.tests.test_sdist.SDistTestCase) -- Traceback (most recent call last): File "D:\Buildslave\3.x.moore-windows\build\lib\tarfile.p

[issue11561] "coverage" of Python regrtest cannot see initial import of libs

Changes by Éric Araujo : Removed file: http://bugs.python.org/file22622/unnamed ___ Python tracker ___ ___ Python-bugs-list mailing list Unsub

[issue11561] "coverage" of Python regrtest cannot see initial import of libs

Éric Araujo added the comment: > I think your suggestions are all good ones, and I have incorporated > them into the file. Great :) You left some commented-out debugging prints. > (But do note that the departures we are now making from Ned's own > copy of the tracer code ­— removing the commen

[issue12681] unittest expectedFailure could take a message argument like skip does

Benjamin Peterson added the comment: Too late I'm afraid as expectedFailure not expectedFailure() is the decorator. -- nosy: +benjamin.peterson ___ Python tracker ___ __

[issue11933] newer() function in dep_util.py mixes up new vs. old files due stat.st_mtime vs stat

Éric Araujo added the comment: Thanks again! -- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker ___

[issue12682] Meaning of 'accepted' resolution as documented in devguide

Brett Cannon added the comment: I say just go ahead and change it. I was probably just thinking how I wish we would use it when I wrote that. -- nosy: +brett.cannon ___ Python tracker __

[issue12683] urlparse.urljoin different behavior for different scheme

New submission from Jasper van den Bosch : urlparse.urljoin successfully joins 'http://localhost/repo1' with a filename, but not 'svn://localhost/repo1' (only scheme different). But the documentation states that the svn: scheme is supported: http://docs.python.org/library/urlparse.html Python

[issue12683] urlparse.urljoin different behavior for different scheme

Changes by Ezio Melotti : -- components: +Library (Lib) -None nosy: +ezio.melotti, orsenthil versions: +Python 3.2, Python 3.3 ___ Python tracker ___

[issue12611] 2to3 crashes when converting doctest using reduce()

Vlada Peric added the comment: Confirmed with the file Aaron linked to. I'm using "2to3-3.2 -w -n -d sympy/ntheory/factor_.py". This is what Python says about itself: Python 3.2 (r32:88445, Jun 8 2011, 16:34:06) [GCC 4.5.1 20101208 [gcc-4_5-branch revision 167585]] on linux2 -- stat

[issue12655] Expose sched.h functions

Roundup Robot added the comment: New changeset 89e92e684b37 by Benjamin Peterson in branch 'default': expose sched.h functions (closes #12655) http://hg.python.org/cpython/rev/89e92e684b37 -- nosy: +python-dev resolution: -> fixed stage: patch review -> committed/rejected status: open

[issue11049] add tests for test.support

Vinay Sajip added the comment: test_forget still doesn't work using my installed Python: test test_support failed -- Traceback (most recent call last): File "/usr/local/lib/python3.3/test/test_support.py", line 62, in test_forget mod = __import__(TESTFN) ImportError: No module named '@tes

[issue12663] ArgumentParser.error writes to stderr not to stdout

Roundup Robot added the comment: New changeset 5ff56995976c by Senthil Kumaran in branch '3.2': Fix closes issue12663 - Correcting the ArgumentParser.error description. Patch by Sandro Tosi. http://hg.python.org/cpython/rev/5ff56995976c New changeset a5b3d5051fc7 by Senthil Kumaran in branch '

[issue12682] Meaning of 'accepted' resolution as documented in devguide

Senthil Kumaran added the comment: I propose that we remove 'accepted' as the meaning seems ambiguous and it is of little practical (/tracking) use. -- nosy: +orsenthil ___ Python tracker _

Re: [issue12677] Turtle, fix right/left rotation orientation

The orientation depends upon the mode, which is explained further down. http://docs.python.org/py3k/library/turtle.html#turtle.left So, the correct fix would be: "would turn clockwise/counterclockwise depending upon the mode." ___ Python-bugs-list maili

[issue11598] missing afxres.h error when building bdist_wininst in Visual Studio 2008 Express

Changes by Ned Deily : -- nosy: +brian.curtin, loewis ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://ma

[issue12681] unittest expectedFailure could take a message argument like skip does

Michael Foord added the comment: Well, expectedFailure could dispatch on the type of the argument, with different behaviour for strings and anything else (presumed to be a function / method). That would be inconsistent with the api for skipping though. (I'm not wild on the message argument to

[issue12682] Meaning of 'accepted' resolution as documented in devguide

R. David Murray added the comment: It is not ambigous. As I said, bugs are fixed, feature requests are accepted. -- ___ Python tracker ___ _

[issue12682] Meaning of 'accepted' resolution as documented in devguide

Senthil Kumaran added the comment: I thought "fixed" was enough. A kind of binary state where we say, okay we have it and otherwise no, we don't. I know for feature requests the terminology does not make sense, but I believe I have worked with bug trackers where fixed was the ultimate state w

[issue12684] profile does not dump stats on exception like cProfile does

New submission from Matt Joiner : Here's a patch that fixes it. -- components: Library (Lib) files: exception-in-profile.patch keywords: patch messages: 141591 nosy: anacrolix priority: normal severity: normal status: open title: profile does not dump stats on exception like cProfile doe

[issue12437] _ctypes.dlopen does not include errno in OSError

Matt Joiner added the comment: Should I just submit a patch for this myself? Can someone confirm the behaviour is incorrect so I don't waste time fixing it? -- ___ Python tracker _

[issue11049] add tests for test.support

Roundup Robot added the comment: New changeset 07d94cf3521e by Eli Bendersky in branch 'default': Issue #11049: fix test_forget on installed Python - add os.curdir to sys.path http://hg.python.org/cpython/rev/07d94cf3521e -- ___ Python tracker

[issue11049] add tests for test.support

Eli Bendersky added the comment: Vinay, I can't reproduce your problem here (tried installing Python and running from there), but I suspect it's a sys.path issue (you don't have os.curdir on it, I do). Please let me know if it works now. -- ___ Pyt

[issue12540] "Restart Shell" command leaves pythonw.exe processes running

Ned Deily added the comment: I've added a couple of review comments to the one Peter already made in Rietveld. Here is an updated patch that addresses all of the comments. I've tested in briefly on Windows and on OS X and it seems to work OK. Eli, if you're OK with it, feel free to commit

[issue12685] The backslash escape doesn't concatenate two strings in one in the with statement

New submission from py.user : >>> with open('/etc/passwd') as f1, \ ... open('/etc/profile) as f2: File "", line 2 open('/etc/profile) as f2: ^ SyntaxError: EOL while scanning string literal >>> >>> with open('/etc/passwd') as f1, open('/etc/profi

[issue12685] The backslash escape doesn't concatenate two strings in one in the with statement

Benjamin Peterson added the comment: Why would you expect that to concatenate strings? You have an unterminated quote? -- nosy: +benjamin.peterson resolution: -> invalid status: open -> closed ___ Python tracker

[issue12685] The backslash escape doesn't concatenate two strings in one in the with statement

py.user added the comment: yes, you're right, this is my mistake -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue7424] segmentation fault in listextend during install

rpointel added the comment: Hi, we have the same problem on OpenBSD (sparc). You could find more information: http://marc.info/?l=openbsd-ports&m=131219537505698 Thanks, Remi. -- ___ Python tracker

[issue12682] Meaning of 'accepted' resolution as documented in devguide

Martin v. Löwis added the comment: The "Accepted" Resolution was copied from SF. I recall seeing documentation on SF for the resolutions, but can't find that anymore. I also recall that on SF, the "accepted" resolution was used to indicate that a patch was accepted (be it a bug fix or a featu

[issue12682] Meaning of 'accepted' resolution as documented in devguide

Changes by Petri Lehtinen : -- nosy: +petri.lehtinen ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mai