[issue13103] copy of an asyncore dispatcher causes infinite recursion

Changes by Ezio Melotti : -- nosy: +giampaolo.rodola, josiahcarlson, stutzbach stage: -> test needed ___ Python tracker ___ ___ Pytho

[issue13102] xml.dom.minidom does not support default namespaces

Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue13104] urllib.request.thishost() returns a garbage value

Changes by Ezio Melotti : -- nosy: +ezio.melotti stage: patch review -> test needed ___ Python tracker ___ ___ Python-bugs-list mailin

[issue13105] Please elaborate on how 2.x and 3.x are different heads

Changes by Ezio Melotti : -- nosy: +ezio.melotti stage: -> needs patch ___ Python tracker ___ ___ Python-bugs-list mailing list Unsub

[issue13106] Incorrect pool.py distributed with Python 2.7 windows 32bit

Changes by Ezio Melotti : -- nosy: +jnoller stage: -> test needed type: -> behavior ___ Python tracker ___ ___ Python-bugs-list mail

[issue9098] MSYS build fails with `S_IXGRP' undeclared

Olivier Refalo added the comment: hum, your patch actually works on MSYS ! ok.. so I am pretty much having the very some issue. Could not find platform dependent libraries Consider setting $PYTHONHOME to [:] Fatal Python error: Py_Initialize: unable to load the file system codec LookupError:

[issue3163] module struct support for ssize_t and size_t

Meador Inge added the comment: Found a few test case nits. Comments in rietveld. -- ___ Python tracker ___ ___ Python-bugs-list maili

[issue13081] Crash in Windows with unknown cause

Brian Curtin added the comment: I tried that script on 2.7 and like it did for you, it just ran until my machine became unusable. On 3.x I think I got a RuntimeError after a while, but I forgot exactly what happened since the machine ended up being hosed later from the 2.7 run. In any event,

[issue13106] Incorrect pool.py distributed with Python 2.7 windows 32bit

New submission from Aaron Staley : The multiprocess/pool.py distributed with the Python 2.7.2 Windows Installer is different from the one distributed with the 64 bit windows installer or source tarball - and is buggy. Specifically, see Pool._terminate_pool: def _terminate_pool(cls, taskq

[issue13101] Module Doc viewer closes when browser window closes on Windows 8

Mark Hammond added the comment: For some reason, IE is struggling to even display the page - it just seems to sit there loading the page without displaying anything, but hitting "stop" then "refresh" usually brings it up. But if you kill IE (which best I can tell can only be done via the tas

[issue13104] urllib.request.thishost() returns a garbage value

Changes by Ned Deily : -- nosy: +orsenthil stage: -> patch review versions: +Python 3.3 ___ Python tracker ___ ___ Python-bugs-list m

[issue12880] ctypes: clearly document how structure bit fields are allocated

Terry J. Reedy added the comment: If I understand correctly, this doc patch would apply to 2.7 and 3.2 also. I have two style comments. I believe "It is important to note that bit field allocation and layout in memory is not defined as a standard, rather its implementation is compiler-specifi

[issue10348] multiprocessing: use SysV semaphores on FreeBSD

Jesse Noller added the comment: Charles and Antoine's votes match my own, therefore closing the bug wont fix -- resolution: -> wont fix status: open -> closed ___ Python tracker __

[issue13105] Please elaborate on how 2.x and 3.x are different heads

New submission from Larry Hastings : It wasn't clear to me after reading the "Forward Porting" section exactly what was going on. Nick Coghlan spelled it out for me in a private email, and suggested that maybe this stuff should be in the devguide proper. Here's some specific stuff that I did

[issue13104] urllib.request.thishost() returns a garbage value

New submission from Deokhwan Kim : There is a minor typo in Lib/urllib/request.py:thishost(). Because of it, the thishost() function is returning a garbage value: >>> import urllib.request >>> urllib.request.thishost() ('XXX.X.XXX.com', ['X.X.XXX.com'], ['123.45.67.89']) It i

[issue13053] Add Capsule migration documentation to "cporting"

Larry Hastings added the comment: New patch based on comments from Ezio Melotti--thanks, Ezio! * capsulethunk.h is now its own file in Doc/includes. * Various minor formatting touchups. * I added some rationale behind the thunked PyCapsule_SetName behavior. -- Added file: http://bugs.p

[issue13061] Decimal module yields incorrect results when Python compiled with clang

Changes by Ned Deily : -- assignee: ronaldoussoren -> ned.deily stage: -> committed/rejected status: pending -> closed title: Decimal module yields incorrect results when Python compiled with llvm -> Decimal module yields incorrect results when Python compiled with clang _

[issue13102] xml.dom.minidom does not support default namespaces

Jeremy Kloth added the comment: Please read the link which you posted. Quoting the second paragraph, second sentence: "Default namespace declarations do not apply directly to attribute names;" and from the third paragraph, third sentence: "The namespace name for an unprefixed attribute name

[issue13081] Crash in Windows with unknown cause

Amorilia added the comment: Quick update: apparently, fixing another seemingly unrelated bug, fixed this crashing issue as well for rlibiez. Here's relevant the commit: https://github.com/amorilia/pyffi/commit/bd7886eefedfce8fb108c4701cf0467e2a707907 Basically, the problem was with multiproce

[issue13103] copy of an asyncore dispatcher causes infinite recursion

New submission from Xavier de Gaye : A regression occurs in python 3.2 when doing a copy of an asyncore dispatcher. $ python3.1 Python 3.1.2 (r312:79147, Apr 4 2010, 17:46:48) [GCC 4.3.2] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> import asyncore, copy

[issue13102] xml.dom.minidom does not support default namespaces

New submission from Glenn Washburn : When using getAttributeNS, attributes with no namespace should be considered as having the default namespace for that scope. See examples in http://www.w3.org/TR/REC-xml-names/#defaulting. Python's xml.dom.minidom will always set the namespace to None for

[issue13101] Module Doc viewer closes when browser window closes on Windows 8

Brian Curtin added the comment: The menu shortcut opens up the following: "C:\Python32\pythonw.exe" "C:\Python32\Tools\scripts\pydocgui.pyw", which is just pydoc.gui() -- ___ Python tracker __

[issue13101] Module Doc viewer closes when browser window closes on Windows 8

Nick Coghlan added the comment: Slight correction, pydoc.gui() is already gone in current hg tip. However, this error may be indicative of an underlying problem with webbrowser.open(url) throwing an exception. -- ___ Python tracker

[issue13101] Module Doc viewer closes when browser window closes on Windows 8

Nick Coghlan added the comment: If that's the app I think it is (pydoc -g), we're probably going to kill it off in 3.3 in favour of the -b option. -- nosy: +ncoghlan ___ Python tracker ___

[issue11956] 3.3 : test_import.py causes 'make test' to fail

Roundup Robot added the comment: New changeset cbda512c6d7f by Charles-François Natali in branch '3.2': Issue #11956: Always skip test_import.test_unwritable_directory when run as http://hg.python.org/cpython/rev/cbda512c6d7f New changeset 971093a75613 by Charles-François Natali in branch 'defa

[issue12880] ctypes: clearly document how structure bit fields are allocated

Vlad Riscutia added the comment: I agree compiler matters for alignment but if you look at PyCField_FromDesc, you will see the layout is pretty much #ifdef MS_WIN32 - #else. Sorry for generalizing, "all" indeed is not the right word. My point is that we should set expectation correctly - VC++

[issue13101] Module Doc viewer closes when browser window closes on Windows 8

New submission from Brian Curtin : Reported by Ryan Wells (v-ry...@microsoft.com) of Microsoft, in reference to a problem with the Module Doc viewer on Windows 8 when using Internet Explorer 10. This was reported on 3.2.2, but it's likely the same on 2.7. Reference #: 70652 Description of the

[issue6715] xz compressor support

Antoine Pitrou added the comment: > > That's because all system-specific functionality that we have really > > depends on system features which just can't be available elsewhere. > > For all functionality that in principle works on all systems, it also > > actually works on all systems for Pyth

[issue6715] xz compressor support

Antoine Pitrou added the comment: > That's because all system-specific functionality that we have really > depends on system features which just can't be available elsewhere. > For all functionality that in principle works on all systems, it also > actually works on all systems for Python. In c

[issue6715] xz compressor support

Martin v. Löwis added the comment: > Not really. xz is becoming a defacto standard under Linux (and perhaps > other free Unices) while I guess it is marginal under Windows. > We have other system-specific functionality, and nobody sees it as a bad > thing. That's because all system-specific fun

[issue13054] sys.maxunicode value after PEP-393

Ezio Melotti added the comment: The buildbot seems happy, so I'm closing this. Antoine already took care of test_bigmem, and Victor opened #13100 for sre_compile. -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed

[issue12880] ctypes: clearly document how structure bit fields are allocated

Meador Inge added the comment: > Look in 'cfield.c' where all of the native alignments Well, not *all* the native alignments, but many of them. -- ___ Python tracker ___ __

[issue6715] xz compressor support

Antoine Pitrou added the comment: > This module is only useful in the standard library if it is available > on all systems. Not really. xz is becoming a defacto standard under Linux (and perhaps other free Unices) while I guess it is marginal under Windows. We have other system-specific functi

[issue12880] ctypes: clearly document how structure bit fields are allocated

Meador Inge added the comment: On Tue, Oct 4, 2011 at 10:21 AM, Vlad Riscutia wrote: > First, I'm saying "toying with the underlying buffer" because none of the > bugs are actual issues of the form "I created this bitfield > structure with Python, passed it to C function but C structure was

[issue6715] xz compressor support

Martin v. Löwis added the comment: Am 04.10.11 19:08, schrieb Antoine Pitrou: > > Antoine Pitrou added the comment: > >>> Based on Amaury's report, I would suggest going forward integrating >>> the xz module for configure-based systems, and letting someone else >>> handle Windows integration la

[issue10348] multiprocessing: use SysV semaphores on FreeBSD

Antoine Pitrou added the comment: Agreed with Charles-François. -- nosy: +pitrou ___ Python tracker ___ ___ Python-bugs-list mailing

[issue10348] multiprocessing: use SysV semaphores on FreeBSD

Charles-François Natali added the comment: -1 IMHO, implementing SysV semaphores would be a step backwards, plus the API is a real pain. I think there's no reason to complicate the code to accomodate such corner cases, especially since the systems that don't support POSIX semaphores will even

[issue7689] Pickling of classes with a metaclass and copy_reg

Nick Coghlan added the comment: Specifically, 2.7.3. A date for that has not yet been set, but somewhere in the December/January time frame is likely. -- ___ Python tracker ___

[issue11956] 3.3 : test_import.py causes 'make test' to fail

Roundup Robot added the comment: New changeset 7697223df6df by Charles-François Natali in branch '3.2': Issue #11956: Skip test_import.test_unwritable_directory on FreeBSD when run as http://hg.python.org/cpython/rev/7697223df6df New changeset 58870fe9a604 by Charles-François Natali in branch '

[issue13100] sre_compile._optimize_unicode() needs a cleanup

New submission from STINNER Victor : The following comment is wrong, except IndexError: # non-BMP characters; XXX now they should work return charset sys.maxunicode != 65535 is now always true in Python 3.3 if sys.maxunicode != 65535: # XXX: negation do

[issue6715] xz compressor support

Antoine Pitrou added the comment: > > Based on Amaury's report, I would suggest going forward integrating > > the xz module for configure-based systems, and letting someone else > > handle Windows integration later if a solution is found. > > -1000. I feel quite strongly that this should not be

[issue7689] Pickling of classes with a metaclass and copy_reg

Antoine Pitrou added the comment: Yes, it will. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue6715] xz compressor support

Martin v. Löwis added the comment: > Based on Amaury's report, I would suggest going forward integrating > the xz module for configure-based systems, and letting someone else > handle Windows integration later if a solution is found. -1000. I feel quite strongly that this should not be added un

[issue6715] xz compressor support

Martin v. Löwis added the comment: > - liblzma can't be compiled by Visual Studio: too many C99 isms, > mostly variables declared in the middle of a block. It's doable for > sure, but it's a lot of work. I'd be in favor of doing so, and then feeding patches upstream. Hopefully, eventually, the

[issue7689] Pickling of classes with a metaclass and copy_reg

Brent Payne added the comment: will the 2.7 patch also be incorporated into a 2.7 release? -- ___ Python tracker ___ ___ Python-bugs-l

[issue13055] Distutils tries to handle null versions but fails

Éric Araujo added the comment: Thanks, will fix it. -- assignee: tarek -> eric.araujo components: +Distutils2 nosy: +alexis versions: +3rd party, Python 3.2, Python 3.3 ___ Python tracker _

[issue12804] "make test" fails on systems without internet access

Éric Araujo added the comment: > Actually, the skip system is already supposed to work for that if used > properly (see test.support.transient_internet()). However, perhaps it > actually doesn't work in all situations. It’s better than that: nearly all tests requiring network access use skips,

[issue6715] xz compressor support

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

[issue13054] sys.maxunicode value after PEP-393

Roundup Robot added the comment: New changeset f39b26ca7f3d by Ezio Melotti in branch 'default': #13054: fix usage of sys.maxunicode after PEP-393. http://hg.python.org/cpython/rev/f39b26ca7f3d -- ___ Python tracker

[issue9035] os.path.ismount on windows doesn't support windows mount points

Brian Curtin added the comment: We can't depend on stuff from pywin32, but we could expose GetVolumePathName ourselves. -- ___ Python tracker ___ ___

[issue9035] os.path.ismount on windows doesn't support windows mount points

Oren Held added the comment: Anything wrong with the following simple approach? (e.g. is it bad to depend on win32file?) def win_ismount(path): import win32file volume_path = win32file.GetVolumePathName(path) return volume_path == path # May have to ignore a trailing backslash -

[issue12880] ctypes: clearly document how structure bit fields are allocated

Vlad Riscutia added the comment: Thanks for the "make patchcheck" tip, I didn't know about that. I will update the patch soon. In the mean time, I want to point out a couple of things: First, I'm saying "toying with the underlying buffer" because none of the bugs are actual issues of the form

[issue3163] module struct support for ssize_t and size_t

Antoine Pitrou added the comment: New patch with cosmetic doc fix. -- Added file: http://bugs.python.org/file23312/struct_nn3.patch ___ Python tracker ___ ___

[issue3163] module struct support for ssize_t and size_t

Antoine Pitrou added the comment: Thanks. I have answered one of your comments, and here is a new patch. -- Added file: http://bugs.python.org/file23311/struct_nn2.patch ___ Python tracker _

[issue3163] module struct support for ssize_t and size_t

Meador Inge added the comment: Mostly LGTM. I have a few comments in rietveld. -- nosy: +meador.inge stage: needs patch -> patch review ___ Python tracker ___ __

[issue12326] Linux 3: code should avoid using sys.platform == 'linux2'

Barry A. Warsaw added the comment: On Oct 04, 2011, at 01:03 PM, Boštjan Mejak wrote: >I have a better idea... Why don't we change the "linux2" string into just >"linux". That way we will never run into this kind of issue, even in the >future when Linux kernel version 4 is going to exist. Any t

[issue12326] Linux 3: code should avoid using sys.platform == 'linux2'

Boštjan Mejak added the comment: I have a better idea... Why don't we change the "linux2" string into just "linux". That way we will never run into this kind of issue, even in the future when Linux kernel version 4 is going to exist. Any thoughts on this? -- nosy: +Retro

[issue3163] module struct support for ssize_t and size_t

Antoine Pitrou added the comment: Here is a patch. -- Added file: http://bugs.python.org/file23310/struct_nn.patch ___ Python tracker ___

[issue13063] test_concurrent_futures failures on Windows: IOError('[Errno 232] The pipe is being closed') on _send_bytes()

STINNER Victor added the comment: > Attached patch. Could you please explain your change in generrmap.c in a comment? For example, just add a reference to this issue. -- ___ Python tracker __

[issue3163] module struct support for ssize_t and size_t

Changes by Stefan Krah : -- nosy: +skrah ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/

[issue3163] module struct support for ssize_t and size_t

Antoine Pitrou added the comment: Ooops, I meant #13098. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: htt

[issue3163] module struct support for ssize_t and size_t

Antoine Pitrou added the comment: #3163 is a duplicate. -- nosy: +pitrou priority: low -> normal stage: patch review -> needs patch ___ Python tracker ___ ___

[issue13098] the struct module should support storage for size_t / Py_ssize_t C types

Changes by Antoine Pitrou : -- resolution: -> duplicate status: open -> closed superseder: -> module struct support for ssize_t and size_t ___ Python tracker ___ __

[issue13099] Sqlite3 & turkish locale

Antoine Pitrou added the comment: > Thanks, Antoine. Should I still try to write a regression test for it? I've had issues writing regression tests for other Turkish locale-related failures (namely, there are other bugs in some glibcs that could cause the test to fail anyway). I'm not sure it's

[issue13099] Sqlite3 & turkish locale

Thomas Kluyver added the comment: Thanks, Antoine. Should I still try to write a regression test for it? -- ___ Python tracker ___ __

[issue13099] Sqlite3 & turkish locale

Antoine Pitrou added the comment: Fixed, thank you. -- nosy: +pitrou resolution: -> fixed stage: test needed -> committed/rejected status: open -> closed versions: +Python 3.3 ___ Python tracker _

[issue13099] Sqlite3 & turkish locale

Roundup Robot added the comment: New changeset 469555867244 by Antoine Pitrou in branch '3.2': Issue #13099: Fix sqlite3.Cursor.lastrowid under a Turkish locale. http://hg.python.org/cpython/rev/469555867244 New changeset 652e2dacbf4b by Antoine Pitrou in branch 'default': Issue #13099: Fix sql

[issue13099] Sqlite3 & turkish locale

Ezio Melotti added the comment: A patch against Lib/sqlite3/test/regression.py would be nice. -- ___ Python tracker ___ ___ Python-bu

[issue13070] segmentation fault in pure-python multi-threaded server

Charles-François Natali added the comment: > Probably. OTOH, not setting the error state when returning NULL is > usually an error (and can result in difficult-to-debug problems), so > let's stay on the safe side. > RuntimeError perhaps. OK, I'll update the patch accordingly. > Does that mean

[issue13070] segmentation fault in pure-python multi-threaded server

Amaury Forgeot d'Arc added the comment: An "unraisable exception" warning will be displayed. -- ___ Python tracker ___ ___ Python-bug

[issue13099] Sqlite3 & turkish locale

Thomas Kluyver added the comment: What form does the test need to be in? There's a script at the redhat bug I linked that demonstrates the issue. Do I need to turn it into a function? A patch for the existing test suite? -- type: behavior -> ___ P

[issue13099] Sqlite3 & turkish locale

Changes by Ezio Melotti : -- nosy: +ezio.melotti stage: -> test needed type: -> behavior ___ Python tracker ___ ___ Python-bugs-list

[issue13070] segmentation fault in pure-python multi-threaded server

Victor Semionov added the comment: > Probably. OTOH, not setting the error state when returning NULL is > usually an error (and can result in difficult-to-debug problems), so > let's stay on the safe side. > > > Furthermore, I'm not sure about what kind of error would make sense here. > > Runt

[issue13098] the struct module should support storage for size_t / Py_ssize_t C types

Mark Dickinson added the comment: See issue #3163. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://m

[issue13063] test_concurrent_futures failures on Windows: IOError('[Errno 232] The pipe is being closed') on _send_bytes()

Changes by Antoine Pitrou : -- components: +Tests nosy: +amaury.forgeotdarc stage: -> patch review type: -> behavior versions: +Python 3.2 ___ Python tracker ___ __

[issue13063] test_concurrent_futures failures on Windows: IOError('[Errno 232] The pipe is being closed') on _send_bytes()

Antoine Pitrou added the comment: I think the solution would be to map ERROR_NO_DATA (232 - "The pipe is being closed") to EPIPE. Attached patch. -- keywords: +patch Added file: http://bugs.python.org/file23309/error_no_data.patch ___ Python tracker

[issue13099] Sqlite3 & turkish locale

New submission from Thomas Kluyver : When using sqlite3 with the Turkish locale, cursor.lastrowid is not accessible after an insert statement if "INSERT" is upper case. I believe that the cause is that the detect_statement_kind function [1] calls the locale-dependent C function tolower(). The

[issue13070] segmentation fault in pure-python multi-threaded server

Antoine Pitrou added the comment: > > Also, is it ok to just return NULL or should the error state also be > > set? > > Well, I'm not sure, that why I made you and Amaury noisy :-) > AFAICT, this is the only case where _check_closed can encounter a NULL > self->writer. Probably. OTOH, not sett

[issue12807] Optimization/refactoring for {bytearray, bytes, unicode}.strip()

Antoine Pitrou added the comment: > The patch no longer applies cleanly. Is there enough interest in this to > justify rebasing? Yes, I think it's worth it. -- title: Optimization/refactoring for {bytearray,bytes,unicode}.strip() -> Optimization/refactoring for {bytearray, bytes, uni

[issue13087] C BufferedReader seek() is inconsistent with UnsupportedOperation for unseekable streams

Antoine Pitrou added the comment: Committed, thanks! -- resolution: -> fixed stage: -> committed/rejected status: open -> closed ___ Python tracker ___ ___

[issue12876] Make Test Error : ImportError: No module named _sha256

Wong Wah Meng added the comment: Hello there, I am encountering more modules/commands that uses the harslib that needs _sha256. I still haven't found a solution. Can anyone shed some lights here whether or not this is related to the way I "include" and "link" the library, or is that the _sha

[issue13087] C BufferedReader seek() is inconsistent with UnsupportedOperation for unseekable streams

Roundup Robot added the comment: New changeset d287f0654349 by Antoine Pitrou in branch '3.2': Issue #13087: BufferedReader.seek() now always raises UnsupportedOperation http://hg.python.org/cpython/rev/d287f0654349 New changeset 0cf38407a3a2 by Antoine Pitrou in branch 'default': Issue #13087:

[issue13098] the struct module should support storage for size_t / Py_ssize_t C types

New submission from Antoine Pitrou : Title says it all. -- components: Library (Lib) messages: 144867 nosy: mark.dickinson, meador.inge, pitrou, skrah priority: normal severity: normal stage: needs patch status: open title: the struct module should support storage for size_t / Py_ssize_t

[issue6715] xz compressor support

Antoine Pitrou added the comment: Based on Amaury's report, I would suggest going forward integrating the xz module for configure-based systems, and letting someone else handle Windows integration later if a solution is found. -- ___ Python tracker

[issue7689] Pickling of classes with a metaclass and copy_reg

Antoine Pitrou added the comment: This is fixed now, thank you! -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed versions: -Python 3.1 ___ Python tracker __

[issue7689] Pickling of classes with a metaclass and copy_reg

Roundup Robot added the comment: New changeset 64053bd79590 by Antoine Pitrou in branch '2.7': Issue #7689: Allow pickling of dynamically created classes when their http://hg.python.org/cpython/rev/64053bd79590 -- ___ Python tracker

[issue7689] Pickling of classes with a metaclass and copy_reg

Roundup Robot added the comment: New changeset 760ac320fa3d by Antoine Pitrou in branch '3.2': Issue #7689: Allow pickling of dynamically created classes when their http://hg.python.org/cpython/rev/760ac320fa3d New changeset 46c026a5ccb9 by Antoine Pitrou in branch 'default': Issue #7689: Allow

[issue7689] Pickling of classes with a metaclass and copy_reg

Antoine Pitrou added the comment: No need, I'll do it myself. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue12156] test_multiprocessing.test_notify_all() timeout (1 hour) on FreeBSD 7.2

STINNER Victor added the comment: > "OSError: [Errno 23] Too many open files in system" Yes, see issue #10348. -- ___ Python tracker ___ ___

[issue7689] Pickling of classes with a metaclass and copy_reg

Craig Citro added the comment: Antoine -- ah, that makes sense. Is that the only blocker? I've let this patch rot on the vine a long time; if so, I'll happily switch `__eq__` back to `__cmp__` and re-post if it'll get submitted. -- ___ Python track

[issue7689] Pickling of classes with a metaclass and copy_reg

Antoine Pitrou added the comment: Craig: I'm talking about the __eq__ version (durban's patch). The __cmp__ version is probably fine. -- ___ Python tracker ___ _

[issue7689] Pickling of classes with a metaclass and copy_reg

Craig Citro added the comment: Antoine -- why do you want to switch "if r" for "if not r"? If we did, the test would just confirm that the unpicked object was of the same type as the original; if we were going to do that, we might as well just replace the whole `__cmp__` function with just `