[issue18049] Re-enable threading test on OSX

Change by Ronald Oussoren : -- pull_requests: +14543 pull_request: https://github.com/python/cpython/pull/14748 ___ Python tracker ___ _

[issue18049] Re-enable threading test on OSX

Ronald Oussoren added the comment: I have a patch for making the stack sizes the same in Issue34264. -- ___ Python tracker ___ ___

[issue18049] Re-enable threading test on OSX

Stéphane Wirtel added the comment: yes, maybe, could you provide a patch ? -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue18049] Re-enable threading test on OSX

Ronald Oussoren added the comment: The patch increases the stack size for threads to slightly less than size as is used for the main thread, see this fragment in configure.ac: # Issue #18075: the default maximum stack size (8MBytes) is too # small for the default recursion limit. Increase

[issue18049] Re-enable threading test on OSX

Stéphane Wirtel added the comment: for my part, I think you should use a #ifdef PY_DEBUG in your code and increase the default size of the stack for the threads, and only if you are on APPLE and PY_DEBUG mode. -- nosy: +matrixise ___ Python tracker

[issue18049] Re-enable threading test on OSX

Ronald Oussoren added the comment: I intend to apply this patch to default (and only that branch) after running the testsuite and with a small change: the stack size will be 0x100 (double it current value) to sync it with the stack size for the main thread, which was recently set to that v

[issue18049] Re-enable threading test on OSX

New submission from Ronald Oussoren: I recently notied that test.test_threading.ThreadingExceptionTests.test_recursion_limit is disabled for debug builds. The attached patch re-enables this test case and fixes the crash be increasing the stack size for new threads. The disadvantage of the