[issue38712] add signal.pidfd_send_signal

[Benjamin Peterson]

Benjamin Peterson  added the comment:


New changeset 7483451577916e693af6d20cf520b2cc7e2174d2 by Benjamin Peterson in 
branch 'master':
closes bpo-38712: Add signal.pidfd_send_signal. (GH-17070)
https://github.com/python/cpython/commit/7483451577916e693af6d20cf520b2cc7e2174d2


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38712] add signal.pidfd_send_signal

[Benjamin Peterson]

Benjamin Peterson  added the comment:

Yes, I thought about that a bit. I choose signal because:

1. signal already has some specialized functions that send signals pthread_kill 
and raise_signal.
2. If we every do want to implement the third parameter of pidfd_send_signal, 
the signal module already defines a siginfo structseq.
3. (least convincing) os/posix has way too much stuff. I generally think that 
we should strive to find a good home for new OS-level functionality that isn't 
the os module.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38712] add signal.pidfd_send_signal

[Nathaniel Smith]

Nathaniel Smith  added the comment:

I guess a bikeshed question is whether it should be `os.pidfd_send_signal` 
(like `os.kill`) or `signal.pidfd_send_signal` (like `signal.pthread_kill`).

I don't actually care either way myself :-)

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38712] add signal.pidfd_send_signal

[Benjamin Peterson]

Change by Benjamin Peterson :


--
keywords: +patch
pull_requests: +16579
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/17070

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38712] add signal.pidfd_send_signal

[Benjamin Peterson]

Change by Benjamin Peterson :


--
nosy: +njs

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38712] add signal.pidfd_send_signal

[Benjamin Peterson]

New submission from Benjamin Peterson :

Add signal.pidfd_send_signal, which will wrap the Linux 5.1 syscall of the same 
name.

--
components: Library (Lib)
messages: 356091
nosy: benjamin.peterson
priority: normal
severity: normal
status: open
title: add signal.pidfd_send_signal
versions: Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com