Revision: 110495 https://trac.macports.org/changeset/110495 Author: blair@macports.org Date: 2013-08-31 11:08:24 -0700 (Sat, 31 Aug 2013) Log Message: ----------- py-zeroc-ice34: fix core dump after Python 2.7.4 upgrade. The unit tests used to successfully pass. $ port -v test py27-zeroc-ice34 ... ... *** running tests 3/25 in /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_python_py-zeroc-ice34/py27-zeroc-ice34/work/Ice-3.4.2/py/test/Ice/adapterDeactivation *** configuration: Default *** test started: 08/31/13 11:02:23 starting Server.py... ok starting Client.py... ok testing stringToProxy... ok testing checked cast... ok creating/destroying/recreating object adapter... ok creating/activating/deactivating object adapter in one operation... ok deactivating object adapter in the server... ok testing whether server is gone... ok unexpected exit status: expected: 0, got -11 Traceback (most recent call last): File "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_python_py-zeroc-ice34/py27-zeroc-ice34/work/Ice-3.4.2/py/test/Ice/adapterDeactivation/run.py", line 23, in <module> TestUtil.clientServerTest() File "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_python_py-zeroc-ice34/py27-zeroc-ice34/work/Ice-3.4.2/scripts/TestUtil.py", line 1185, in clientServerTest clientProc.waitTestSuccess() File "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_python_py-zeroc-ice34/py27-zeroc-ice34/work/Ice-3.4.2/scripts/Expect.py", line 540, in waitTestSuccess test(self.exitstatus, exitstatus) File "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_python_py-zeroc-ice34/py27-zeroc-ice34/work/Ice-3.4.2/scripts/Expect.py", line 525, in test assert False AssertionError ('test in /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_python_py-zeroc-ice34/py27-zeroc-ice34/work/Ice-3.4.2/py/test/Ice/adapterDeactivation failed with exit status', 256) $ /usr/bin/gdb /opt/local/bin/python2.7 /cores/core.92512 GNU gdb 6.3.50-20050815 (Apple version gdb-1824) (Wed Feb 6 22:51:23 UTC 2013) Copyright 2004 Free Software Foundation, Inc. GDB is free software, covered by the GNU General Public License, and you are welcome to change it and/or distribute copies of it under certain conditions. Type "show copying" to see the conditions. There is absolutely no warranty for GDB. Type "show warranty" for details. This GDB was configured as "x86_64-apple-darwin"...Reading symbols for shared libraries ... done Reading symbols for shared libraries . done Reading symbols for shared libraries ....................................... done #0 subtype_dealloc (self=0x103d1bb90) at Objects/typeobject.c:955 955 ++ tstate->trash_delete_nesting; (gdb) bt #0 subtype_dealloc (self=0x103d1bb90) at Objects/typeobject.c:955 #1 0x0000000103db4c9b in virtual thunk to IcePy::LoggerWrapper::~LoggerWrapper() (this=<value temporarily unavailable, due to optimizations>) at Logger.h:23 #2 0x00007fff8b3fc7c8 in __cxa_finalize () #3 0x00007fff8b3fc652 in exit () #4 0x000000010390853d in handle_system_exit () at Python/pythonrun.c:1774 #5 0x0000000103908169 in PyErr_PrintEx (set_sys_last_vars=1) at Python/pythonrun.c:1156 #6 0x0000000103907b42 in PyErr_Print [inlined] () at /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_lang_python27/python27/work/Python-2.7.5/Python/pythonrun.c:1059 #7 0x0000000103907b42 in PyRun_SimpleFileExFlags (fp=<value temporarily unavailable, due to optimizations>, filename=<value temporarily unavailable, due to optimizations>, closeit=1, flags=0x7fff6343b3d0) at Python/pythonrun.c:947 #8 0x000000010391b8c0 in Py_Main (argc=<value temporarily unavailable, due to optimizations>, argv=0x7fff6343b428) at Modules/main.c:640 #9 0x000000010383df24 in _mh_execute_header () $ diff -U5 Python-2.7.{3,4}/Objects/typeobject.c --- Python-2.7.3/Objects/typeobject.c 2012-04-09 16:07:34.000000000 -0700 +++ Python-2.7.4/Objects/typeobject.c 2013-04-06 07:02:39.000000000 -0700 @@ -887,10 +896,11 @@ static void subtype_dealloc(PyObject *self) { PyTypeObject *type, *base; destructor basedealloc; + PyThreadState *tstate = PyThreadState_GET(); /* Extract the type; we expect it to be a heap type */ type = Py_TYPE(self); assert(type->tp_flags & Py_TPFLAGS_HEAPTYPE); @@ -936,12 +946,14 @@ /* UnTrack and re-Track around the trashcan macro, alas */ /* See explanation at end of function for full disclosure */ PyObject_GC_UnTrack(self); ++_PyTrash_delete_nesting; + ++ tstate->trash_delete_nesting; Py_TRASHCAN_SAFE_BEGIN(self); --_PyTrash_delete_nesting; + -- tstate->trash_delete_nesting; /* DO NOT restore GC tracking at this point. weakref callbacks * (if any, and whether directly here or indirectly in something we * call) may trigger GC, and if self is tracked at that point, it * will look like trash to GC and GC will try to delete self again. */ This core dump doesn't occur with Ice 3.5.0's Python bindings, so doing a diff between 3.4.2 and 3.5.0, I found code to clean up the logger using atexit(). Modified Paths: -------------- trunk/dports/python/py-zeroc-ice34/Portfile Added Paths: ----------- trunk/dports/python/py-zeroc-ice34/files/patch-fix-python-2.7.4-core-dump.diff Modified: trunk/dports/python/py-zeroc-ice34/Portfile =================================================================== --- trunk/dports/python/py-zeroc-ice34/Portfile 2013-08-31 17:44:01 UTC (rev 110494) +++ trunk/dports/python/py-zeroc-ice34/Portfile 2013-08-31 18:08:24 UTC (rev 110495) @@ -5,7 +5,7 @@ name py-zeroc-ice34 version 3.4.2 -revision 4 +revision 5 set branch [join [lrange [split ${version} .] 0 1] .] categories-append devel maintainers blair @@ -40,7 +40,8 @@ rmd160 7ce680a4eb5fa9d0bb6f8b8910e267dfc2373d75 patchfiles patch-py.config.Make.rules.Darwin.diff \ patch-py.modules.IcePy.Communicator.cpp.diff \ - patch-add-thrift-support.diff + patch-add-thrift-support.diff \ + patch-fix-python-2.7.4-core-dump.diff platforms darwin python.versions 25 26 27 Added: trunk/dports/python/py-zeroc-ice34/files/patch-fix-python-2.7.4-core-dump.diff =================================================================== --- trunk/dports/python/py-zeroc-ice34/files/patch-fix-python-2.7.4-core-dump.diff (rev 0) +++ trunk/dports/python/py-zeroc-ice34/files/patch-fix-python-2.7.4-core-dump.diff 2013-08-31 18:08:24 UTC (rev 110495) @@ -0,0 +1,81 @@ +diff -ru ../Ice-3.4.2.orig/py/modules/IcePy/Init.cpp ./py/modules/IcePy/Init.cpp +--- ../Ice-3.4.2.orig/py/modules/IcePy/Init.cpp 2011-06-15 12:44:00.000000000 -0700 ++++ ./py/modules/IcePy/Init.cpp 2013-08-31 10:43:39.000000000 -0700 +@@ -29,6 +29,7 @@ + using namespace IcePy; + + extern "C" PyObject* Ice_registerTypes(PyObject*, PyObject*); ++extern "C" PyObject* IcePy_cleanup(PyObject*); + + static PyMethodDef methods[] = + { +@@ -70,6 +71,8 @@ + PyDoc_STR(STRCAST("internal function")) }, + { STRCAST("loadSlice"), reinterpret_cast<PyCFunction>(IcePy_loadSlice), METH_VARARGS, + PyDoc_STR(STRCAST("loadSlice(cmd) -> None")) }, ++ { STRCAST("cleanup"), reinterpret_cast<PyCFunction>(IcePy_cleanup), METH_NOARGS, ++ PyDoc_STR(STRCAST("internal function")) }, + { 0, 0 } /* sentinel */ + }; + +@@ -150,3 +153,13 @@ + return; + } + } ++ ++extern "C" ++PyObject* ++IcePy_cleanup(PyObject* /*self*/) ++{ ++ cleanupLogger(); ++ ++ Py_INCREF(Py_None); ++ return Py_None; ++} +diff -ru ../Ice-3.4.2.orig/py/modules/IcePy/Logger.cpp ./py/modules/IcePy/Logger.cpp +--- ../Ice-3.4.2.orig/py/modules/IcePy/Logger.cpp 2011-06-15 12:44:00.000000000 -0700 ++++ ./py/modules/IcePy/Logger.cpp 2013-08-31 10:33:03.000000000 -0700 +@@ -400,6 +400,15 @@ + return true; + } + ++void ++IcePy::cleanupLogger() ++{ ++ // ++ // Python is about to exit; we need to remove the wrapper around the process logger. ++ // ++ Ice::setProcessLogger(0); ++} ++ + PyObject* + IcePy::createLogger(const Ice::LoggerPtr& logger) + { +diff -ru ../Ice-3.4.2.orig/py/modules/IcePy/Logger.h ./py/modules/IcePy/Logger.h +--- ../Ice-3.4.2.orig/py/modules/IcePy/Logger.h 2011-06-15 12:44:00.000000000 -0700 ++++ ./py/modules/IcePy/Logger.h 2013-08-31 10:32:30.000000000 -0700 +@@ -41,6 +41,8 @@ + + bool initLogger(PyObject*); + ++void cleanupLogger(); ++ + // + // Create a Python object that delegates to a C++ implementation. + // +diff -ru ../Ice-3.4.2.orig/py/python/Ice.py ./py/python/Ice.py +--- ../Ice-3.4.2.orig/py/python/Ice.py 2011-06-15 12:44:00.000000000 -0700 ++++ ./py/python/Ice.py 2013-08-31 10:44:36.000000000 -0700 +@@ -53,6 +53,12 @@ + sys.setdlopenflags(_dlopenflags) + + # ++# Give the extension an opportunity to clean up before a graceful exit. ++# ++import atexit ++atexit.register(IcePy.cleanup) ++ ++# + # Add some symbols to the Ice module. + # + ObjectPrx = IcePy.ObjectPrx