[issue16500] Add an 'atfork' module

2017-05-22 Thread Antoine Pitrou
Antoine Pitrou added the comment: I've posted https://github.com/python/cpython/pull/1715 which adds a single `os.register_at_fork` function to allow registering at-fork handlers. Comments welcome. -- stage: -> patch review versions: +Python 3.7 -Python 3.5

[issue16500] Add an 'atfork' module

2017-05-22 Thread Antoine Pitrou
Changes by Antoine Pitrou : -- pull_requests: +1805 ___ Python tracker ___ ___

[issue16500] Add an 'atfork' module

2016-05-29 Thread A. Jesse Jiryu Davis
Changes by A. Jesse Jiryu Davis : -- nosy: +emptysquare ___ Python tracker ___ ___

[issue16500] Add an 'atfork' module

2016-05-29 Thread Марк Коренберг
Марк Коренберг added the comment: Also I have to add, maybe you also add automatic file descriptor closing on fork. We already have O_CLOEXEC, we have no O_CLOFORK. So, maybe it useful to add such flag ? -- ___ Python tracker

[issue16500] Add an 'atfork' module

2016-05-29 Thread Марк Коренберг
Changes by Марк Коренберг : -- nosy: +mmarkk ___ Python tracker ___ ___

[issue16500] Add an 'atfork' module

2016-03-29 Thread Robert Cope
Changes by Robert Cope : -- nosy: +rpcope1 ___ Python tracker ___ ___ Python-bugs-list

[issue16500] Add an 'atfork' module

2015-10-19 Thread Ionel Cristian Mărieș
Changes by Ionel Cristian Mărieș : -- nosy: +ionelmc ___ Python tracker ___ ___

[issue16500] Add an 'atfork' module

2015-05-26 Thread Yury Selivanov
Changes by Yury Selivanov yseliva...@gmail.com: -- nosy: +yselivanov ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___ ___ Python-bugs-list

[issue16500] Add an 'atfork' module

2013-12-26 Thread xupeng
Changes by xupeng xup...@xupeng.me: -- nosy: +xupeng ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___ ___ Python-bugs-list mailing list

[issue16500] Add an 'atfork' module

2013-12-22 Thread Antoine Pitrou
Changes by Antoine Pitrou pit...@free.fr: -- versions: +Python 3.5 -Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___ ___

[issue16500] Add an 'atfork' module

2013-11-01 Thread Richard Oudkerk
Richard Oudkerk added the comment: Given PEP 446 (fds are now CLOEXEC by default) I prepared an updated patch where the fork lock is undocumented and subprocess no longer uses the fork lock. (I did not want to encourage the mixing of threads with fork() without exec() by exposing the fork

[issue16500] Add an 'atfork' module

2013-11-01 Thread Charles-François Natali
Charles-François Natali added the comment: Given PEP 446 (fds are now CLOEXEC by default) I prepared an updated patch where the fork lock is undocumented and subprocess no longer uses the fork lock. (I did not want to encourage the mixing of threads with fork() without exec() by exposing

[issue16500] Add an 'atfork' module

2013-11-01 Thread STINNER Victor
STINNER Victor added the comment: The PEP 446 does not offer any warranty on the atomicity on clearing the inheritable flag. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___

[issue16500] Add an 'atfork' module

2013-11-01 Thread Richard Oudkerk
Richard Oudkerk added the comment: It is a recent kernel and does support pipe2(). After some debugging it appears that a pipe handle created in Popen.__init__() was being leaked to a forked process, preventing Popen.__init__() from completing before the forked process did. Previously the

[issue16500] Add an 'atfork' module

2013-10-21 Thread Christian Heimes
Christian Heimes added the comment: Richard, do you have time to get your patch ready for 3.4? -- assignee: christian.heimes - ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___

[issue16500] Add an 'atfork' module

2013-10-21 Thread Richard Oudkerk
Richard Oudkerk added the comment: Richard, do you have time to get your patch ready for 3.4? Yes. But we don't seem to have concensus on how to handle exceptions. The main question is whether a failed prepare callback should prevent the fork from happenning, or just be printed.

[issue16500] Add an 'atfork' module

2013-10-21 Thread Dwayne Litzenberger
Dwayne Litzenberger added the comment: The main question is whether a failed prepare callback should prevent the fork from happenning Yes, I think an exception should prevent the fork from happening. It's fail-safe for the PRNG case (you can guarantee that a fork won't occur without

[issue16500] Add an 'atfork' module

2013-10-21 Thread Christian Heimes
Christian Heimes added the comment: +1 for exception prevents fork -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___ ___ Python-bugs-list

[issue16500] Add an 'atfork' module

2013-10-21 Thread Charles-François Natali
Charles-François Natali added the comment: I have a couple random remarks: - now that FDs are non-inheritable by default, fork locks around subprocess and multiprocessing shouldn't be necessary anymore? What other use cases does the fork-lock have? - the current implementation keeps

[issue16500] Add an 'atfork' module

2013-10-21 Thread STINNER Victor
STINNER Victor added the comment: now that FDs are non-inheritable by default, fork locks around subprocess and multiprocessing shouldn't be necessary anymore? What other use cases does the fork-lock have? Well, on Windows, it's still not possible to inherit only one handle. If you mark

[issue16500] Add an 'atfork' module

2013-10-21 Thread Richard Oudkerk
Richard Oudkerk added the comment: - now that FDs are non-inheritable by default, fork locks around subprocess and multiprocessing shouldn't be necessary anymore? What other use cases does the fork-lock have? CLOEXEC fds will still be inherited by forked children. - the current

[issue16500] Add an 'atfork' module

2013-10-21 Thread Charles-François Natali
Charles-François Natali added the comment: Richard Oudkerk added the comment: - now that FDs are non-inheritable by default, fork locks around subprocess and multiprocessing shouldn't be necessary anymore? What other use cases does the fork-lock have? CLOEXEC fds will still be inherited

[issue16500] Add an 'atfork' module

2013-10-21 Thread Antoine Pitrou
Antoine Pitrou added the comment: One of the main use cases for atfork hooks would be the numerous stdlib objects which have locks (or locks themselves): most of such objects have arbitrary lifetimes (e.g. logging, events, open files, etc). The risk of leak is IMO much greater. Well it is

[issue16500] Add an 'atfork' module

2013-10-21 Thread Charles-François Natali
Charles-François Natali added the comment: Well it is customary for callback-based APIs to hold strong references to their callbacks. If a library wants to avoid leaks, it should register a single callback which will then walk the current live resources and protect them. I guess that the

[issue16500] Add an 'atfork' module

2013-10-16 Thread Aaron Iles
Changes by Aaron Iles aaron.i...@gmail.com: -- nosy: +aliles ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___ ___ Python-bugs-list mailing

[issue16500] Add an 'atfork' module

2013-09-30 Thread Dwayne Litzenberger
Changes by Dwayne Litzenberger dl...@dlitz.net: -- nosy: +DLitz ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___ ___ Python-bugs-list

[issue16500] Add an 'atfork' module

2013-08-15 Thread Antoine Pitrou
Changes by Antoine Pitrou pit...@free.fr: -- nosy: +neologix ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___ ___ Python-bugs-list mailing

[issue16500] Add an 'atfork' module

2013-01-14 Thread STINNER Victor
STINNER Victor added the comment: Might make sense to put this in atexit.atfork() to avoid small-module inflation? It sounds strange to mix at exit and at fork in the same module. Both are very different. 2013/1/13 Arfrever Frehtes Taifersar Arahesis rep...@bugs.python.org: Changes by

[issue16500] Add an 'atfork' module

2013-01-14 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: On 13.01.2013 00:37, STINNER Victor wrote: By the way, OpenSSL expects that its PRNG is reseed somehow (call RNG_add) after a fork. I wrote a patch for OpenSSL, but I don't remember if I sent it to OpenSSL.

[issue16500] Add an 'atfork' module

2013-01-14 Thread Antoine Pitrou
Antoine Pitrou added the comment: It sounds strange to mix at exit and at fork in the same module. Both are very different. That's true. The sys module would probably be the right place for both functionalities. -- ___ Python tracker

[issue16500] Add an 'atfork' module

2013-01-13 Thread Georg Brandl
Georg Brandl added the comment: Might make sense to put this in atexit.atfork() to avoid small-module inflation? -- nosy: +georg.brandl ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500

[issue16500] Add an 'atfork' module

2013-01-13 Thread Arfrever Frehtes Taifersar Arahesis
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com: -- nosy: +Arfrever ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16500 ___

[issue16500] Add an 'atfork' module

2013-01-12 Thread STINNER Victor
STINNER Victor added the comment: The tempfile module has a specialized RNG that re-initialized the RNG after fork() by comparing os.getpid() to an instance variable every time the RNG is accessed. The check can be replaced with an afterfork callback. By the way, OpenSSL expects that its PRNG