`add_signal_handler` support for all platforms
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 77.2k
- Fork
- 35.9k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
Feature or enhancement
Proposal:
I'm currently writing this to support signals on Windows and Linux:
loop = asyncio.get_running_loop()
def shutdown(*args):
// some shutdown sequence
if sys.platform == "win32":
for sig in (signal.SIGINT, signal.SIGTERM):
signal.signal(sig, shutdown)
else:
for sig in (signal.SIGINT, signal.SIGTERM):
loop.add_signal_handler(sig, shutdown)
Works great, but why would signal.signal be supported on Linux and Windows, but asyncio's AbstractEventLoop via add_signal_handler only on Linux? Can't this be platform agnostic?
add_signal_handler would raise the following on Windows:
[...]\Python\Python313\Lib\asyncio\events.py", line 596, in add_signal_handler
raise NotImplementedError
Has this already been discussed elsewhere?
No response given
Links to previous discussion of this feature:
No response
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia con asyncio.AbstractEventLoop.add_signal_handler e con il NotImplementedError mostrato alla riga 596 di Python313/Lib/asyncio/events.py, quindi confrontalo con signal.signal su Windows e Linux. Il lavoro è completato quando è stato stabilito se add_signal_handler può fornire il comportamento multipiattaforma richiesto e il supporto della piattaforma risultante è stato documentato o implementato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- operating-systems
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 25/100