[issue5628] TextIOWrapper fails with SystemError when reading HTTPResponse

2009-03-31 Thread Jeremy Hylton
Jeremy Hylton added the comment: I just wanted to mention that the current head of py3k returns an http.client.HTTPResponse and not a urllib.respone.addinfourl. That doesn't mean it is the right thing to pass to TextIOWrapper. It's an instance of

[issue5578] unqualified exec in class body

2009-04-01 Thread Jeremy Hylton
Jeremy Hylton added the comment: Why did it change from 2.5 to 2.6? I'm not sure that the change makes any sense. (Dreading the answer that I changed it...) -- ___ Python tracker <http://bugs.python.org/i

[issue4291] Allow Division of datetime.timedelta Objects

2009-04-14 Thread Jeremy Banks
Jeremy Banks added the comment: Redundant with #2706 and others. -- nosy: -belopolsky, haypo, marketdickinson status: open -> closed ___ Python tracker <http://bugs.python.org/iss

[issue2531] float compared to decimal is silently incorrect.

2009-04-24 Thread Jeremy Dunck
Jeremy Dunck added the comment: I'm interested. I just had already said my peace and didn't know my prior interest wasn't being counted. The patch uploaded by dmmartins seemed good to me. I'm probably biased, since

[issue2746] ElementTree ProcessingInstruction uses character entities in content

2009-06-07 Thread Jeremy Thurgood
Changes by Jeremy Thurgood : -- nosy: +jerith ___ Python tracker <http://bugs.python.org/issue2746> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue6230] ElementTree.Element and cElementTree.Element have slightly different repr

2009-06-07 Thread Jeremy Thurgood
Changes by Jeremy Thurgood : -- nosy: +jerith ___ Python tracker <http://bugs.python.org/issue6230> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue6231] ElementInclude may drop text

2009-06-07 Thread Jeremy Thurgood
Changes by Jeremy Thurgood : -- nosy: +jerith ___ Python tracker <http://bugs.python.org/issue6231> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue6232] Improve test coverage of ElementTree and cElementTree

2009-06-10 Thread Jeremy Thurgood
Changes by Jeremy Thurgood : -- nosy: +jerith ___ Python tracker <http://bugs.python.org/issue6232> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue8254] write a configure command

2010-05-26 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue8254> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue8254] write a configure command

2010-07-17 Thread Jeremy Kloth
Jeremy Kloth added the comment: WRT #5, it would be possible to use section headers within the configure.cache file to indicate a particular "platform build". Very much like how 4Suite's `config` command stored the cache file. -- ___

[issue8254] write a configure command

2010-07-20 Thread Jeremy Kloth
Jeremy Kloth added the comment: for #1 instead of importing `build`/`install` commands for their options, why not define the options only on the `configure` command? The B/I commands could just "lookup" their needed options from the configure command. On that note, how does the B/

[issue8254] write a configure command

2010-07-20 Thread Jeremy Kloth
Jeremy Kloth added the comment: How would the build or install command get the options in this case: python setup.py configure --prefix=$HOME build install Remember that commands can be chained (and that commands can run other commands). I would like to be able to do (and can do right now

[issue9335] LC_CTYPE system setting not respected by setlocale()

2010-07-22 Thread Jeremy Kloth
Jeremy Kloth added the comment: Note that this behavior is only present when running IDLE. Python command-line does not show this oddity. -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue9

[issue28231] zipfile does not support pathlib

2017-01-05 Thread Jeremy Freeman
Jeremy Freeman added the comment: This also affects python 3.5 on Windows and OSX. Python 3.5.2 (default, Sep 21 2016, 15:07:18) [GCC 4.2.1 Compatible Apple LLVM 7.3.0 (clang-703.0.31)] on darwin Type "help", "copyright", "credits" or "license" for more i

[issue28231] zipfile does not support pathlib

2017-01-05 Thread Jeremy Freeman
Jeremy Freeman added the comment: OK, I understand. How can I help get this issue fixed? 1) review the patch? 2) update the docs to reflect the patch? 3) find the other uses of pathlib in the zipfile module? 4) something else ... I am a longtime user of python but a first time contributor but

[issue28231] zipfile does not support pathlib

2017-01-10 Thread Jeremy Freeman
Jeremy Freeman added the comment: I have reviewed the code and docs for the public API that should take a pathlib.Path object: - zipfile.is_zipfile(filename) - filename - zipfile.ZipFile(file) - file - ZipFile.extract(member, path=None) - path - ZipFile.extractall(path=None) - path

[issue29344] sock_recv not detected a coroutine

2017-01-22 Thread Jeremy Bustamante
New submission from Jeremy Bustamante: Documemtation says sock_recv is a coroutine https://docs.python.org/3.6/library/asyncio-eventloop.html#low-level-socket-operations But following code says it isnt: import asyncio loop = asyncio.get_event_loop() asyncio.iscoroutinefunction(loop.sock_recv

[issue29530] Windows buildbots broken by the migration to GitHub (meta issue)

2017-02-11 Thread Jeremy Kloth
Jeremy Kloth added the comment: After ensuring that Git is available to the buildbot service, the initial update fails due to the existing build directory containing the Mercurial checkout. After manually removing the contents of the build directories, the update task successfully completes

[issue29530] Windows buildbots broken by the migration to GitHub (meta issue)

2017-02-11 Thread Jeremy Kloth
Jeremy Kloth added the comment: Tests are now failing due to end-of-line (?) differences. I did a quick reading of the devguide, but didn't see any mention of the settings to be used for Git on Windows wrt core.autocrlf. -- ___ Python tr

[issue27371] Runaway memory consumption using update()

2016-06-22 Thread Jeremy Blow
New submission from Jeremy Blow: Hi folks, While going about running tkinter's update() call within a coroutine/asyncio event loop in lieu of using mainloop(), I came across odd memory behavior on MacOS where the cpython process gradually increases memory consumption. Suspicious that

[issue27371] Runaway memory consumption using tkinter update()

2016-06-23 Thread Jeremy Blow
Jeremy Blow added the comment: That seems like a good idea, will do so and report back findings. Hey, good meeting you at PyCon, by the way. Met you at the freelance open space. :-) -- ___ Python tracker <http://bugs.python.org/issue27

[issue27371] Runaway memory consumption using tkinter update()

2016-06-23 Thread Jeremy Blow
Jeremy Blow added the comment: Confirmed the problem is with Tk/MacOS. Running a simple TCL script yielded the same behavior in terms of memory. The hunt goes elsewhere... Thank you R. David for the suggestion. ``` package require Tk while 1 { update } ``` # MacOS 10.9.4 /Users/jeremy

[issue27410] DLL hijacking vulnerability in Python 3.5.2 installer

2016-06-29 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue27410> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue27533] release GIL in nt._isdir

2016-07-16 Thread Jeremy Spiegel
New submission from Jeremy Spiegel: In Modules/posixmodule.c, the function os__isdir_impl calls GetFileAttributesA/GetFileAttributesW without releasing the GIL. This is problematic since it can result in disk or network I/O, depending on the filesystem. -- components: Library (Lib

[issue27533] release GIL in nt._isdir

2016-07-16 Thread Jeremy Spiegel
Changes by Jeremy Spiegel : Added file: http://bugs.python.org/file43757/_isdir.2.7.patch ___ Python tracker <http://bugs.python.org/issue27533> ___ ___ Python-bugs-list m

[issue27533] release GIL in nt._isdir

2016-07-16 Thread Jeremy Spiegel
Changes by Jeremy Spiegel : Added file: http://bugs.python.org/file43758/_isdir.2.7.patch ___ Python tracker <http://bugs.python.org/issue27533> ___ ___ Python-bugs-list m

[issue27533] release GIL in nt._isdir

2016-07-16 Thread Jeremy Spiegel
Changes by Jeremy Spiegel : Removed file: http://bugs.python.org/file43757/_isdir.2.7.patch ___ Python tracker <http://bugs.python.org/issue27533> ___ ___ Python-bug

[issue27781] Change sys.getfilesystemencoding() on Windows to UTF-8

2016-08-17 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue27781> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue17360] Regular expressions on mmap'd files can overflow

2013-03-05 Thread Jeremy Archer
New submission from Jeremy Archer: I've recently been having problems with the Python regular expressions module when mmap'ing large (> 2^32 bits) on Linux. Steps to reproduce: https://gist.github.com/fatlotus/5094575 I've been able to replicate it on Ubuntu Linux (u

[issue17480] pyvenv should be installed someplace more obvious on Windows

2013-03-19 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue17480> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue17065] Fix sporadic buildbot failures for test_winreg

2013-04-21 Thread Jeremy Kloth
Jeremy Kloth added the comment: Not to sound needy, but could the patch be looked into being integrated soon? This problem had only occurred once or twice a month however it has caused failures three times just in the last week. -- ___ Python

[issue17065] Fix sporadic buildbot failures for test_winreg

2013-04-21 Thread Jeremy Kloth
Jeremy Kloth added the comment: Thank you! There are no failures due to the patch and now its just a wait and see if test_winreg will misbehave again. -- ___ Python tracker <http://bugs.python.org/issue17

[issue17846] Building Python on Windows - Supplementary info

2013-04-27 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue17846> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue18002] AMD64 Windows7 SP1 3.x buildbot: compilation of _ssl module fails, the build fails

2013-05-18 Thread Jeremy Kloth
Jeremy Kloth added the comment: The build of OpenSSL was failing due to an incomplete external check-in of OpenSSL (missing the cached assembler files). Martin has since updated the external and I have refreshed the OpenSSL exports on the buildbot. It is no longer failing to compile, but

[issue22895] regression introduced by the fix for issue #22462

2014-11-18 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue22895> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22462] Modules/pyexpat.c violates PEP 384

2014-11-18 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue22462> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22919] Update PCBuild for VS 2015

2014-11-23 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue22919> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue23154] MSVC 2013 Express needlessly rebuilds code

2015-01-03 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue23154> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue23437] Make user scripts directory versioned on Windows

2015-02-10 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue23437> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue23462] All os.exec*e variants crash on Windows

2015-02-14 Thread Jeremy Nicola
New submission from Jeremy Nicola: On Windows 7, using python 3.4.2 32 bits MSVCv.1600 or 3.4.1 64 bits, all the os.exec*e variants crash: os.execle('C:/Python34/python.exe','Python.exe','-V',{}) os.execve('C:/Python34/python.exe',['python.exe&#

[issue23462] All os.exec*e variants crash on Windows

2015-02-14 Thread Jeremy Nicola
Jeremy Nicola added the comment: You should read: os.execv('C:/Python34/python.exe',['python.exe','-V']) os.execlp('C:/Python34/python.exe','python.exe','-V') os.execvp('

[issue23472] Setup locales on buildbots

2015-02-17 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue23472> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue25220] Enhance and refactor test.regrtest (convert regrtest.py to a package)

2015-09-23 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue25220> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue25289] test_strptime hangs sometimes on AMD64 Windows7 SP1 3.x buildbot

2015-10-01 Thread Jeremy Kloth
Jeremy Kloth added the comment: The system is a dedicated Quad CPU @2.66GHz with 8GB RAM and running the tests on an SSD. I doubt it is the hardware. Please note, it is the only 64-bit Windows buildbot AND the only multi-core Windows buildbot. So it catches lots of things that most others

[issue26201] Faster type checking in listobject.c

2016-01-25 Thread Jeremy Kloth
Jeremy Kloth added the comment: Added review -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue26201> ___ ___ Python-bugs-list mailing list Unsub

[issue26207] test_distutils fails on "AMD64 Windows7 SP1 3.x" buildbot

2016-01-26 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/issue26207> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue26207] test_distutils fails on "AMD64 Windows7 SP1 3.x" buildbot

2016-01-28 Thread Jeremy Kloth
Jeremy Kloth added the comment: My buildbot updated to VS2015.1 successfully. test_distutils is passing again. As to the bug title, it should mention the incorrect DLL version error, not the /LTCG *warning*. That's what I get for just doing a quick scan of the com

[issue26207] test_distutils fails on "AMD64 Windows7 SP1 3.x" buildbot

2016-01-28 Thread Jeremy Kloth
Jeremy Kloth added the comment: I'm upgrading the VS installation on my buildbot currently so this should be a non-issue going forward. However, I think it would be good to add/change the bug title to include something about the /LTCG error so as to enable better searching when develope

[issue26239] distutils link-objects is not normalized

2016-01-29 Thread Jeremy Fergason
New submission from Jeremy Fergason: When giving setup.cfg a link-objects argument build fails with an error message about cannot concatenate string with list. Need to add the following line to: Lib/distutils/command/build_ext.py self.ensure_string_list('link_objects') See

[issue26624] Windows hangs in call to CRT setlocale()

2016-03-23 Thread Jeremy Kloth
New submission from Jeremy Kloth: My Windows BuildBot (http://buildbot.python.org/all/buildslaves/kloth-win64) is hanging in calls to the CRT function setlocale() as determined by attaching to the hung test process in Visual Studio. This has been happening occasionally (every tenth+ build

[issue26624] Windows hangs in call to CRT setlocale()

2016-03-23 Thread Jeremy Kloth
Jeremy Kloth added the comment: Oh, yes. It even occasionally happens on 2.7. The oldest occurrence I can dig up is May/June of 2013 (for 2.7). -- ___ Python tracker <http://bugs.python.org/issue26

[issue26624] Windows hangs in call to CRT setlocale()

2016-03-23 Thread Jeremy Kloth
Jeremy Kloth added the comment: >From the UCRT sources: // Deadlock Avoidance: When a new thread is created in the process, we // create a new PTD for the thread. The PTD initialization function is // called under the loader lock. This initialization function will a

[issue26624] Windows hangs in call to CRT setlocale()

2016-03-24 Thread Jeremy Kloth
Jeremy Kloth added the comment: The lastest set of updates were installed on 1/28. Visual Studio does have Update 1. I'm doing another round of updates now, although nothing related to VC 2015 or Visual Studio, however. -- ___ Python tr

[issue26658] test_os fails when run on Windows ramdisk

2016-03-28 Thread Jeremy Kloth
New submission from Jeremy Kloth: The Win32JunctionTests class fails when the test suite is run on an ImDisk[1]_ virtual disk. The junctions are created successfully, however os.stat() fails on them (winerror 123). os.lstat() does succeed. I'm inclined to believe that this is a bug i

[issue26624] Windows hangs in call to CRT setlocale()

2016-03-30 Thread Jeremy Kloth
Jeremy Kloth added the comment: I would have responded sooner, but add me to the list of those not receiving tracker notifications... My buildbot is running Windows 7 with all updates (last checked 3/29) and VS2015 Update 1. ucrtbase: 10.0.10586.9 (11/19/2015) ucrtbased

[issue26658] test_os fails when run on Windows ramdisk

2016-03-30 Thread Jeremy Kloth
Jeremy Kloth added the comment: To reproduce: P:\python-default>PCBuild\amd64\python_d.exe Python 3.6.0a0 (default:708beeb65026, Mar 30 2016, 08:50:27) [MSC v.1900 64 bit (AMD64)] on win32 Type "help", "copyright", "credits" or "license" f

[issue26624] Windows hangs in call to CRT setlocale()

2016-03-30 Thread Jeremy Kloth
Jeremy Kloth added the comment: After attempting to find the installer for ucrtbased.dll on my dev machine and reading various blog posts on the Visual C++ Team Blog and hunting through several KB articles, I've come across what I think is needed for an up-to-date Universal CRT.

[issue26658] test_os fails when run on Windows ramdisk

2016-03-30 Thread Jeremy Kloth
Jeremy Kloth added the comment: I'm fine with the tests for CreateFunction failing for an ImDisk virtual drive, however something needs to be changed with the test to not remove the test directory on tearDown(). Changing it to use a temporary directory to link against is a workaround

[issue26624] Windows hangs in call to CRT setlocale()

2016-04-12 Thread Jeremy Kloth
Jeremy Kloth added the comment: It seems that the updated UCRT debug runtime has indeed solved the issue. I suggest that this issue remains open pending an update to the devguide for required settings for installing VS2015 with the updated runtime (see msg262672). I have no idea if the VC

[issue26820] Prevent uses of format string based PyObject_Call* that do not produce tuple required by docs

2016-04-21 Thread Jeremy Kloth
Jeremy Kloth added the comment: IMHO, this is a documentation bug with PyObject_CallMethod. The change to its documentation to differ from PyObject_CallFunction was changed back in 2004. It should have been updated then to reflect the already well-entrenched behavior of those 2 (at the time

[issue23207] logging.basicConfig does not validate keyword arguments

2015-03-17 Thread Jeremy Goss
Jeremy Goss added the comment: The argument validation in basicConfig has introduced another problem. I cannot pass in an optional filename/filemode argument pair. Previously, I passed in an optional logfile argument from ArgumentParser (which could be None). Now, I get a ValueError because

[issue27806] 2.7 32-bit builds fail on future releases of OS X due to dependency on deleted header file

2016-09-15 Thread Jeremy Sequoia
Jeremy Sequoia added the comment: AvailabilityMacros.h is super deprecated and should not be used by anything current. Availability.h was added as replacement in 10.5. __MAC_OS_X_VERSION_MAX_ALLOWED should be checked instead of defined (DEPRECATED_IN_MAC_OS_X_VERSION_10_12_AND_LATER) You

[issue27806] 2.7 32-bit builds fail on future releases of OS X due to dependency on deleted header file

2016-09-15 Thread Jeremy Sequoia
Jeremy Sequoia added the comment: Correction (bugfix and Tiger-compat): #if __has_include() #include #define APPLE_SUPPORTS_QUICKTIME (__MAC_OS_X_VERSION_MAX_ALLOWED < 101200) && !__LP64__ #else #define APPLE_SUPPORTS_QUICKTIME !__LP

[issue27806] 2.7 32-bit builds fail on future releases of OS X due to dependency on deleted header file

2016-09-16 Thread Jeremy Sequoia
Jeremy Sequoia added the comment: Thanks, yep that looks much nicer to me. Containing the core logic at a central choke point makes maintenance much easier. I'll work to get this into MacPorts. -- ___ Python tracker <http://bugs.py

[issue27806] 2.7 32-bit builds fail on future releases of OS X due to dependency on deleted header file

2016-09-16 Thread Jeremy Sequoia
Jeremy Sequoia added the comment: sys/cdefs.h (on new enough darwin) does this for such cases: #ifndef __has_include #define __has_include(x) 0 #endif but of course that isn't present in sys/cdefs.h on older SDKs, so you can just shove that above the check and it should fallback the wa

[issue28224] Compilation warnings on Windows: export 'PyInit_xx' specified multiple times

2016-09-21 Thread Jeremy Kloth
Jeremy Kloth added the comment: This is a known bug when compiling 64-bit extension modules. http://bugs.python.org/issue9709 http://bugs.python.org/issue16779 http://bugs.python.org/issue20166 -- nosy: +jkloth ___ Python tracker <h

[issue27806] 2.7 32-bit builds fail on macOS 10.12 Sierra due to dependency on deleted header file QuickTime/QuickTime.h

2016-10-17 Thread Jeremy Sequoia
Jeremy Sequoia added the comment: QuickTime/QuickTime.h was deprecated long ago. python fails to compile if QuickTime/QuickTime.h isn't present. QuickTime/QuickTime.h was removed in Sierra, causing the build to fail on Sierra. This patch fixes that issue. > On Oct 17, 2016,

[issue28554] Windows: _socket module fails to compile on "AMD64 Windows7 SP1 3.x" buildbot "because "AlwaysCreate" was specified"

2016-10-29 Thread Jeremy Kloth
Jeremy Kloth added the comment: Steve's guess wrt the _pth file being the cause was spot on. The buildbot is back on track with successful results. -- nosy: +jkloth ___ Python tracker <http://bugs.python.org/is

[issue42969] pthread_exit & PyThread_exit_thread from PyEval_RestoreThread etc. are harmful

2021-09-15 Thread Jeremy Maitin-Shepard
Jeremy Maitin-Shepard added the comment: Another possible resolution would to simply make threads that attempt to acquire the GIL after Python starts to finalize hang (i.e. sleep until the process exits). Since the GIL can never be acquired again, this is in some sense the simplest way to

[issue42969] pthread_exit & PyThread_exit_thread from PyEval_RestoreThread etc. are harmful

2021-09-16 Thread Jeremy Maitin-Shepard
Jeremy Maitin-Shepard added the comment: Regarding your suggestion of adding a hook like `Py_SetThreadExitCallback`, it seems like there are 4 plausible behaviors that such a callback may implement: 1. Abort the process immediately with an error. 2. Exit immediately with the original exit

[issue42969] pthread_exit & PyThread_exit_thread from PyEval_RestoreThread etc. are harmful

2021-09-16 Thread Jeremy Maitin-Shepard
Jeremy Maitin-Shepard added the comment: Regarding your suggestion of banning daemon threads: I happened to come across this bug not because of daemon threads but because of threads started by C++ code directly that call into Python APIs. The solution I am planning to implement is to add

[issue42969] pthread_exit & PyThread_exit_thread from PyEval_RestoreThread etc. are harmful

2021-09-20 Thread Jeremy Maitin-Shepard
Jeremy Maitin-Shepard added the comment: It looks like the `_thread` module does not actually expose `PyThread_exit_thread` --- the similarly named `thread_PyThread_exit_thread` just raises SystemExit. >From a search in the codebase, it appears `PyThread_exit_thread` is currently &g

[issue42969] pthread_exit & PyThread_exit_thread from PyEval_RestoreThread etc. are harmful

2021-09-22 Thread Jeremy Maitin-Shepard
Change by Jeremy Maitin-Shepard : -- keywords: +patch pull_requests: +26916 stage: -> patch review pull_request: https://github.com/python/cpython/pull/28525 ___ Python tracker <https://bugs.python.org/issu

[issue42969] pthread_exit & PyThread_exit_thread from PyEval_RestoreThread etc. are harmful

2021-09-22 Thread Jeremy Maitin-Shepard
Jeremy Maitin-Shepard added the comment: I suppose calling `Py_Initialize`, `Py_FinalizeEx`, then `Py_Initialize` again, then `Py_FinalizeEx` again in an embedding application, was already not particularly well supported, since it would leak memory. However, with this change it also leaks

[issue42969] pthread_exit & PyThread_exit_thread from PyEval_RestoreThread etc. are harmful

2021-09-23 Thread Jeremy Maitin-Shepard
Jeremy Maitin-Shepard added the comment: In general, I view hanging threads as the least bad thing to do when faced with either acquiring the GIL or not returning at all. There is a lot of existing usage of Python that currently poses a risk of random crashes and memory corruption while

[issue42969] pthread_exit & PyThread_exit_thread from PyEval_RestoreThread etc. are harmful

2021-09-24 Thread Jeremy Maitin-Shepard
Jeremy Maitin-Shepard added the comment: To be clear, the problem I'm trying to address here is not specific to embedding Python in a C++ application. In fact the issue came to my attention while using Python directly, but loading an extension module that was written in C++ usin

[issue42969] pthread_exit & PyThread_exit_thread from PyEval_RestoreThread etc. are harmful

2021-09-26 Thread Jeremy Maitin-Shepard
Jeremy Maitin-Shepard added the comment: Yes, I would agree that the new APIs are a useful addition regardless of the PyThread_exit_thread change. As far as the proposed `Py_SetThreadExitCallback` that seems like a fine thing for applications to use, as long as it doesn't impac

<    1   2   3   4   5   6