[issue3645] readline module Crashs on OpenBSD/amd64

2008-09-02 Thread Gregory P. Smith
Changes by Gregory P. Smith <[EMAIL PROTECTED]>: -- keywords: -needs review ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-

[issue3645] readline module Crashs on OpenBSD/amd64

2008-09-02 Thread Gregory P. Smith
Gregory P. Smith <[EMAIL PROTECTED]> added the comment: committed to trunk (2.6) in r66179. This should be back ported to release25-maint and automagically merged into py3k. can someone with OpenBSD confirm that this has indeed fixed the problem? if so i'll do the 25 backport and mark it as

[issue2975] VS8 include dirs grow without bound

2008-09-02 Thread Jim Kleckner
Jim Kleckner <[EMAIL PROTECTED]> added the comment: Yes, much better. I should have read into the interior of the discussion rather than just the edges... Thanks! ___ Python tracker <[EMAIL PROTECTED]> _

[issue3756] re.escape() does not work with bytes()

2008-09-02 Thread Andrew McNamara
Changes by Andrew McNamara <[EMAIL PROTECTED]>: Added file: http://bugs.python.org/file11352/re_escape-patch ___ Python tracker <[EMAIL PROTECTED]> ___

[issue2975] VS8 include dirs grow without bound

2008-09-02 Thread Scott Dial
Scott Dial <[EMAIL PROTECTED]> added the comment: This patch shouldn't have been applied as it is. The definition of "removeDuplicates" is both poorly-named, not exactly correct, and redundant (as there is already a "normalize_and_reduce_paths") for performing this fix on PATH. Jim, you acknowl

[issue3637] 2to3 refactoring

2008-09-02 Thread Benjamin Peterson
Benjamin Peterson <[EMAIL PROTECTED]> added the comment: Gregory P. Smith reviewed the change on Rietveld, and the patch was applied in r66173. -- resolution: -> accepted status: open -> closed ___ Python tracker <[EMAIL PROTECTED]>

[issue3756] re.escape() does not work with bytes()

2008-09-02 Thread Andrew McNamara
Andrew McNamara <[EMAIL PROTECTED]> added the comment: Will do, although I'm slightly concerned that my "bytes" version of the function is about 50% slower than the "str" version. I can see why, I just can't think of a way to do it any faster. There's an inherent asymetry in bytes type that di

[issue3697] "Fatal Python error: Cannot recover from stack overflow" on Windows buildbots

2008-09-02 Thread Antoine Pitrou
Changes by Antoine Pitrou <[EMAIL PROTECTED]>: -- nosy: +loewis ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-list mailing

[issue2975] VS8 include dirs grow without bound

2008-09-02 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc <[EMAIL PROTECTED]> added the comment: Applied both patches (a bit differently for the equal sign) as r66171. -- resolution: -> fixed status: open -> closed ___ Python tracker <[EMAIL PROTECTED]>

[issue1638033] Add httponly to Cookie module

2008-09-02 Thread Antoine Pitrou
Antoine Pitrou <[EMAIL PROTECTED]> added the comment: Well, I'm sorry but this a feature request and must be delayed for 2.7/3.1, since 2.6/3.0 are now in the release candidate phase. :-( (as for the patch, it would be nice if it added an unit test for the new feature) -- nosy: +pitrou

[issue1638033] Add httponly to Cookie module

2008-09-02 Thread Matt Chisholm
Matt Chisholm <[EMAIL PROTECTED]> added the comment: Any progress on this? This patch is extremely straightforward (only three lines of code), and should not break existing code. The HttpOnly extension to cookies is now supported by IE, Firefox 3.0, and Opera. This article explains why HttpOn

[issue3444] add warnings for intra-package imports

2008-09-02 Thread Barry A. Warsaw
Barry A. Warsaw <[EMAIL PROTECTED]> added the comment: Well, this would be nice, but it doesn't seem serious enough to block the release. -- priority: release blocker -> critical ___ Python tracker <[EMAIL PROTECTED]>

[issue3637] 2to3 refactoring

2008-09-02 Thread Benjamin Peterson
Changes by Benjamin Peterson <[EMAIL PROTECTED]>: Added file: http://bugs.python.org/file11350/after_gps_review.patch ___ Python tracker <[EMAIL PROTECTED]> ___ ___

[issue3760] PEP 3121 --- PyType_Copy is missing

2008-09-02 Thread Benjamin Peterson
Changes by Benjamin Peterson <[EMAIL PROTECTED]>: -- assignee: -> loewis nosy: +loewis priority: -> high ___ Python tracker <[EMAIL PROTECTED]> ___ __

[issue3761] urllib.request and urllib.response cannot handle HTTP1.1 chunked encoding

2008-09-02 Thread Chris Leow
New submission from Chris Leow <[EMAIL PROTECTED]>: Hi, fairly new to Python, so not sure if this is something you want as a behaviour or not: urllib.response object when fetching an HTTP1.1 page does not transparently handle "Transfer-Encoding": "chunked", and I think it should. You can view s

[issue3759] test_asyncore.py leaks handle

2008-09-02 Thread Hirokazu Yamamoto
Hirokazu Yamamoto <[EMAIL PROTECTED]> added the comment: Thanks, fixed in r66162(trunk), r66163(py3k) -- status: open -> closed ___ Python tracker <[EMAIL PROTECTED]> ___ _

[issue3759] test_asyncore.py leaks handle

2008-09-02 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc <[EMAIL PROTECTED]> added the comment: Tested on Linux: each call to test.test_asyncore.test_main() opens two file descriptors (shown in /proc//fd). The patch corrects this. -- keywords: -needs review nosy: +amaury.forgeotdarc resolution: -> accepted _

[issue3759] test_asyncore.py leaks handle

2008-09-02 Thread Hirokazu Yamamoto
Changes by Hirokazu Yamamoto <[EMAIL PROTECTED]>: -- keywords: +easy, needs review ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python

[issue1251] ssl module doesn't support non-blocking handshakes

2008-09-02 Thread Senthil
Senthil <[EMAIL PROTECTED]> added the comment: Yes Janssen, I checked again and found it implemented in both trunk (py26) and py3k. All the tests pass as well. However, in one of my testcases for issue1424152, where I expected the timeout to happen for do_handshake(), it did not take effect. I

[issue3760] PEP 3121 --- PyType_Copy is missing

2008-09-02 Thread Paul Pogonyshev
New submission from Paul Pogonyshev <[EMAIL PROTECTED]>: PEP 3121 at python.org mentions PyType_Copy(). However, it doesn't seem to be present in SVN version and there is no apparent replacement. Please clarify how types should be created for different module instances --- if just sharing is fi

[issue3419] multiprocessing module is racy

2008-09-02 Thread Jesse Noller
Jesse Noller <[EMAIL PROTECTED]> added the comment: r66161 on py3k merges forward the fix for this ___ Python tracker <[EMAIL PROTECTED]> ___ ___

[issue3729] SystemError on calling len() if __len__() doesn't return an int

2008-09-02 Thread Hagen Fürstenau
Changes by Hagen Fürstenau <[EMAIL PROTECTED]>: Removed file: http://bugs.python.org/file11308/len_check2.diff ___ Python tracker <[EMAIL PROTECTED]> ___ __

[issue3729] SystemError on calling len() if __len__() doesn't return an int

2008-09-02 Thread Hagen Fürstenau
Changes by Hagen Fürstenau <[EMAIL PROTECTED]>: Removed file: http://bugs.python.org/file11307/len_check.diff ___ Python tracker <[EMAIL PROTECTED]> ___ ___

[issue3729] SystemError on calling len() if __len__() doesn't return an int

2008-09-02 Thread Hagen Fürstenau
Hagen Fürstenau <[EMAIL PROTECTED]> added the comment: There seems to be a pronouncement now (http://mail.python.org/pipermail/python-3000/2008-September/014692.html), so I'm attaching a patch which clips to sys.maxsize and includes Amaury's suggestions. Added file: http://bugs.python.org/file11

[issue3758] "make check" suggest a testing target under GNU coding standards

2008-09-02 Thread Brett Cannon
Brett Cannon <[EMAIL PROTECTED]> added the comment: On Tue, Sep 2, 2008 at 7:17 AM, Benjamin Peterson <[EMAIL PROTECTED]> wrote: > > Benjamin Peterson <[EMAIL PROTECTED]> added the comment: > > Brett, how about "patchcheck"? > Maybe. I will see if any inspiration comes to me in the near future.

[issue3419] multiprocessing module is racy

2008-09-02 Thread Jesse Noller
Jesse Noller <[EMAIL PROTECTED]> added the comment: Looks like Mark's patch was submitted as part of r66115 by Ben accidentally (as part of reverting r66114). I can confirm this patch resolves the incref issue as-is. I've run test_multiprocessing in a loop for about an hour now with no failure

[issue1441530] socket read() can cause MemoryError in Windows

2008-09-02 Thread Iain MacKay
Iain MacKay <[EMAIL PROTECTED]> added the comment: Knowing that the large read provokes the problem enables one to write this simple workaround by subclassing IMAP4 without patching the library: maxRead = 100 class MySSL (imaplib.IMAP4_SSL): def read (self, n): #print "..Attempting to

[issue3660] reference leaks in 3.0

2008-09-02 Thread Antoine Pitrou
Changes by Antoine Pitrou <[EMAIL PROTECTED]>: -- keywords: +needs review ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-lis

[issue2690] Precompute range length

2008-09-02 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: Given the problems with it, I'm dropping this to normal priority and punting to 3.1. (the release blocker status was just temporary to ensure we got a decision on it before rc1 - it previously didn't have a priority set) -- keywords:

[issue2690] Precompute range length

2008-09-02 Thread Guido van Rossum
Guido van Rossum <[EMAIL PROTECTED]> added the comment: I wonder if we shouldn't hold off on this. It's a substantial amount of new code. I'd be fine with it going into 3.0.1 since it's pure optimization (IIUC!). But right now we want burn-in, not new features. __

[issue2690] Precompute range length

2008-09-02 Thread Antoine Pitrou
Antoine Pitrou <[EMAIL PROTECTED]> added the comment: By the way, why is this release blocker? do we have performance numbers? ___ Python tracker <[EMAIL PROTECTED]> ___ ___

[issue3081] Py_(X)SETREF macros

2008-09-02 Thread Jesús Cea Avión
Changes by Jesús Cea Avión <[EMAIL PROTECTED]>: -- nosy: +jcea ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-list mailing l

[issue2690] Precompute range length

2008-09-02 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: v2 of my updated patch attached to fix the issue Antoine noted. Also gets rid of some tab-instead-of-spaces indenting issues in the file, and avoids hardcoding PyRange_Type when creating new instances. However, the patch still has issues, as c

[issue3720] segfault in for loop with evil iterator

2008-09-02 Thread Amaury Forgeot d'Arc
Changes by Amaury Forgeot d'Arc <[EMAIL PROTECTED]>: Removed file: http://bugs.python.org/file11317/next-nevernull.patch ___ Python tracker <[EMAIL PROTECTED]> ___

[issue3720] segfault in for loop with evil iterator

2008-09-02 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc <[EMAIL PROTECTED]> added the comment: Oops, sorry. I have too many files opened at the same time. Here is an updated patch. I removed all the "assert(PyIter_Check(it))", our evil iterator used to make them fail anyway in debug mode; and in the case of a null function pointe

[issue2486] Decimal slowdown in 3.0 due to str/unicode changes

2008-09-02 Thread Facundo Batista
Facundo Batista <[EMAIL PROTECTED]> added the comment: Can not get into this now, but I'll be able to deal with this in some weeks... -- assignee: -> facundobatista ___ Python tracker <[EMAIL PROTECTED]> ___

[issue3720] segfault in for loop with evil iterator

2008-09-02 Thread Armin Rigo
Armin Rigo <[EMAIL PROTECTED]> added the comment: Maybe the file 'next-nevernull.patch' is not complete? I don't see any change in the definition of PyIter_Check(). ___ Python tracker <[EMAIL PROTECTED]>

[issue2690] Precompute range length

2008-09-02 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: Good catch Antoine (I missed that in Alexander's patch) - working on that now. ___ Python tracker <[EMAIL PROTECTED]> ___ ___

[issue3719] platform.py: _syscmd_file() can't handle target path with space or special shell character

2008-09-02 Thread jfdp
jfdp <[EMAIL PROTECTED]> added the comment: To respond to a couple questions: Adding the double quotes fixed the issue I had -- but I did very limited testing. The "-b" option is not support by 'file' on Solaris. ___ Python tracker <[EMAIL PROTECTED]>

[issue2690] Precompute range length

2008-09-02 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: My initial patch had a few problems - I removed it and uploaded a corrected version. Added file: http://bugs.python.org/file11346/issue2690-range-sequence-ncoghlan.diff ___ Python tracker <[EMAIL PROTECTED]

[issue2690] Precompute range length

2008-09-02 Thread Nick Coghlan
Changes by Nick Coghlan <[EMAIL PROTECTED]>: Removed file: http://bugs.python.org/file11345/issue2690-range-sequence-ncoghlan.diff ___ Python tracker <[EMAIL PROTECTED]> ___ _

[issue2690] Precompute range length

2008-09-02 Thread Antoine Pitrou
Antoine Pitrou <[EMAIL PROTECTED]> added the comment: It looks like your range_subscript() forgets to compute the length field... ___ Python tracker <[EMAIL PROTECTED]> ___

[issue2690] Precompute range length

2008-09-02 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: This issue was missing a priority setting. Alexander's range-sequence patch still applies cleanly to the Py3k branch, and "make test" still runs correctly after applying it. As Alexander notes above, range_contains does still need slightly bet

[issue3759] test_asyncore.py leaks handle

2008-09-02 Thread Hirokazu Yamamoto
New submission from Hirokazu Yamamoto <[EMAIL PROTECTED]>: asyncore.file_wrapper() dups passed handle, so original handle must be closed. -- components: Tests files: test_asyncore.patch keywords: patch messages: 72343 nosy: ocean-city severity: normal status: open title: test_asyncore.py

[issue2486] Decimal slowdown in 3.0 due to str/unicode changes

2008-09-02 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: Looks like we'll be living with the slowdown for 3.0, so marking this as a high priority item for 3.1. (Given that the API doesn't change, I wonder if this could be included in a 3.0.1 release?) -- priority: -> high versions: +Python

[issue2975] VS8 include dirs grow without bound

2008-09-02 Thread Jim Kleckner
Jim Kleckner <[EMAIL PROTECTED]> added the comment: Any hope for these two patches being applied? ___ Python tracker <[EMAIL PROTECTED]> ___ ___

[issue2975] VS8 include dirs grow without bound

2008-09-02 Thread Jim Kleckner
Jim Kleckner <[EMAIL PROTECTED]> added the comment: Here is the equals sign fix as a separate patch file. Added file: http://bugs.python.org/file11343/equalsInEnv.patch ___ Python tracker <[EMAIL PROTECTED]>

[issue3758] "make check" suggest a testing target under GNU coding standards

2008-09-02 Thread Benjamin Peterson
Benjamin Peterson <[EMAIL PROTECTED]> added the comment: Brett, how about "patchcheck"? -- assignee: -> brett.cannon nosy: +benjamin.peterson, brett.cannon ___ Python tracker <[EMAIL PROTECTED]>

[issue3589] Misleading names for multiprocessing "convenience" functions

2008-09-02 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: Not so much "too complicated" as "potentially touches a lot of code and not something I want to fiddle with this close to release". As I noted on python-dev, it's actually a change that can easily be handled through the normal API deprecation p

[issue3758] "make check" suggest a testing target under GNU coding standards

2008-09-02 Thread Ralph Corderoy
New submission from Ralph Corderoy <[EMAIL PROTECTED]>: A new target, "check", has been added to Makefile for 2.6. It runs some tests on the source code that are intended to check there's nothing wrong before preparing a patch. http://svn.python.org/view/python/trunk/Makefile.pre.in?rev=61528&v

[issue1868] threading.local doesn't free attrs when assigning thread exits

2008-09-02 Thread Christian Heimes
Christian Heimes <[EMAIL PROTECTED]> added the comment: Amaury Forgeot d'Arc wrote: > Maybe __dict__ should be special-cased in local_getattro()? With the patch it's also no longer possible to get a list of attribute names. +1 for special casing __dict__ in getattro and setattro. setattr(local,

[issue1868] threading.local doesn't free attrs when assigning thread exits

2008-09-02 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc <[EMAIL PROTECTED]> added the comment: With the patch, subclasses of threading.local seem to have an empty __dict__: import threading class MyLocal(threading.local): pass l = MyLocal() l.x = 2 l.__dict__ # returns {} Maybe __dict__ should be special-cased in local_ge

[issue3617] Add MS EULA to the list of third-party licenses in the Windows installer

2008-09-02 Thread Marc-Andre Lemburg
Marc-Andre Lemburg <[EMAIL PROTECTED]> added the comment: Raising priority since this needs to be resolved prior to the final release of Python 2.6/3.0. Regarding finding the eula.txt in the VS2008 installation, there doesn't appear to be a generic way. The eula.txt is stored in a folder named a

[issue1868] threading.local doesn't free attrs when assigning thread exits

2008-09-02 Thread Antoine Pitrou
Antoine Pitrou <[EMAIL PROTECTED]> added the comment: The patch is basically fine with me. While reviewing it I've found out that threading.local doesn't have cyclic garbage collecting enabled. I've opened a new issue for it in #3757. -- nosy: +pitrou ___

[issue3757] threading.local doesn't support cyclic garbage collecting

2008-09-02 Thread Antoine Pitrou
New submission from Antoine Pitrou <[EMAIL PROTECTED]>: tp_traverse and tp_clear on threading.local are defined, but the Py_TPFLAGS_HAVE_GC flag is not set. As a result, cycles are not collected: >>> import threading, weakref >>> o = threading.local() >>> class X(object): pass ... >>> x = X() >

[issue3733] Adding bin and Scripts folder into PATH

2008-09-02 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc <[EMAIL PROTECTED]> added the comment: See the long discussion in issue3561. -- nosy: +amaury.forgeotdarc resolution: -> duplicate status: open -> closed superseder: -> Windows installer should add Python and Scripts directories to the PATH environment variable _

[issue3719] platform.py: _syscmd_file() can't handle target path with space or special shell character

2008-09-02 Thread Hirokazu Yamamoto
Hirokazu Yamamoto <[EMAIL PROTECTED]> added the comment: OK, I understand. And I must apologizes about previous my patch. fileout = _architecture_split(output)[1:] should not be deleted. It was should be changed to fileout = _architecture_split(output) Thank you.

[issue3719] platform.py: _syscmd_file() can't handle target path with space or special shell character

2008-09-02 Thread Marc-Andre Lemburg
Marc-Andre Lemburg <[EMAIL PROTECTED]> added the comment: Checked in as r66145 and r66146. -- status: open -> closed ___ Python tracker <[EMAIL PROTECTED]> ___

[issue3719] platform.py: _syscmd_file() can't handle target path with space or special shell character

2008-09-02 Thread Marc-Andre Lemburg
Marc-Andre Lemburg <[EMAIL PROTECTED]> added the comment: On 2008-09-02 12:55, Hirokazu Yamamoto wrote: > Hirokazu Yamamoto <[EMAIL PROTECTED]> added the comment: > > Well, if python was installed into directory named > "foo ELF boo", python will say wrong architecture. > import platform >

[issue3719] platform.py: _syscmd_file() can't handle target path with space or special shell character

2008-09-02 Thread Hirokazu Yamamoto
Hirokazu Yamamoto <[EMAIL PROTECTED]> added the comment: Well, if python was installed into directory named "foo ELF boo", python will say wrong architecture. >>> import platform >>> platform.architecture() ('32bit', 'ELF') Is it not good to use "file -b" option? __

[issue3747] Fix caching in ABCMeta.__subclasscheck__

2008-09-02 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: 2.6 fixed in r66144, merged to 3.0 in r66147 -- status: open -> closed ___ Python tracker <[EMAIL PROTECTED]> ___ __

[issue3719] platform.py: _syscmd_file() can't handle target path with space or special shell character

2008-09-02 Thread Marc-Andre Lemburg
Marc-Andre Lemburg <[EMAIL PROTECTED]> added the comment: Updated the Python versions. -- versions: +Python 2.5, Python 2.6, Python 3.0 -Python 2.4 ___ Python tracker <[EMAIL PROTECTED]> _

[issue3719] platform.py: _syscmd_file() can't handle target path with space or special shell character

2008-09-02 Thread Marc-Andre Lemburg
Marc-Andre Lemburg <[EMAIL PROTECTED]> added the comment: Using subprocess is only possible conditionally, ie. if available. Please note that the module is intended to be usable with multiple Python versions and must at least support Python 2.1. We could consider an updated patch for 2.7 and 3.1

[issue3748] platform.architecture() prints bogus message on windows

2008-09-02 Thread Hirokazu Yamamoto
Hirokazu Yamamoto <[EMAIL PROTECTED]> added the comment: Well, should I backport this to release25-maint branch? (If accepted, issue3719 as well) ___ Python tracker <[EMAIL PROTECTED]> ___

[issue3719] platform.py: _syscmd_file() can't handle target path with space or special shell character

2008-09-02 Thread Hirokazu Yamamoto
Hirokazu Yamamoto <[EMAIL PROTECTED]> added the comment: [off topic] I used cygwin to create this patch, but platform.architecture()[1] becomes empty string because sys.executable in cygwin points to directory. (at least before installing) Python 2.6b3+ (trunk:66142M, Sep 2 2008, 17:09:46) [GCC

[issue3719] platform.py: _syscmd_file() can't handle target path with space or special shell character

2008-09-02 Thread Hirokazu Yamamoto
Hirokazu Yamamoto <[EMAIL PROTECTED]> added the comment: Can you try this patch? 1. used "file -b" option to eliminate file path. Otherwide, re.split won't work because file path will contain space like this. (I hope -b option can be used anywhere "file" command exists) a b/python.exe: MS-DOS e

[issue3352] Deficiencies in multiprocessing/threading API

2008-09-02 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: Thanks! ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-list mailing list Unsubscr

[issue1160] Medium size regexp crashes python

2008-09-02 Thread Adi
Adi <[EMAIL PROTECTED]> added the comment: Is there any progress on this bug? I am currently considering maintaining a branch of 2.5.2 which includes the patch suggested by effbot. -- nosy: +adi ___ Python tracker <[EMAIL PROTECTED]>

[issue3548] subprocess.pipe function

2008-09-02 Thread Legoll Vincent
Legoll Vincent <[EMAIL PROTECTED]> added the comment: On Mon, Sep 1, 2008 at 5:13 PM, Amaury Forgeot d'Arc <[EMAIL PROTECTED]> wrote: > Amaury Forgeot d'Arc <[EMAIL PROTECTED]> added the comment: > > Vincent, > GPL licenced code is incompatible with the inclusion into python. > And if I am correc