search

ericsnowcurrently / multi-core-python

Enabling CPython multi-core parallelism via subinterpreters.
BSD 3-Clause "New" or "Revised" License
245 stars 6 forks source link

Crash with subinterpreters and Py_NewInterpreter() #56

Closed ericsnowcurrently closed 4 years ago

ericsnowcurrently commented 4 years ago

https://bugs.python.org/issue30060

jakirkham commented 4 years ago

This appears to be a Python 2.7 issue only, right?

ericsnowcurrently commented 4 years ago

Hmm, if it is 2.7-only then there's nothing to do. :) It would be worth double-checking. Given how much the runtime initialization/finalization implementation has changed since 2.7, I wouldn't be surprised if it was no longer a problem.

ericsnowcurrently commented 4 years ago

Good catch, BTW.

jakirkham commented 4 years ago

Sure. Happy to help.

It's probably worth noting they were building Python 2.7 with VS 2015, which is not officially supported AFAIK. Additionally they were patching it.

So this may be far enough off the beaten path such that it isn't worth worrying about. Happy to defer to your judgement though.

ericsnowcurrently commented 4 years ago

Yeah, let's not worry about it unless there is a more clearly relevant report. :)