Skip to content

ext/pcntl simplification.#13975

Open
devnexen wants to merge 1 commit intophp:masterfrom
devnexen:pcntl_signal_simplification
Open

ext/pcntl simplification.#13975
devnexen wants to merge 1 commit intophp:masterfrom
devnexen:pcntl_signal_simplification

Conversation

@devnexen
Copy link
Copy Markdown
Member

assuming pcntl is an unix-only extension and they all support the siginfo_t type,
we re using exclusively this more advanced api.

@devnexen devnexen marked this pull request as ready for review April 16, 2024 17:31
@devnexen devnexen requested a review from kocsismate as a code owner April 16, 2024 17:31
@staabm
Copy link
Copy Markdown
Contributor

staabm commented Apr 16, 2024

assuming pcntl is an unix-only

I am aware of this limitation, but was wondering whether the pcntl api could be implemented on windows in the future?

@devnexen
Copy link
Copy Markdown
Member Author

A subset of it is possible I think.

@Girgias
Copy link
Copy Markdown
Member

Girgias commented Apr 6, 2026

Friendly ping to rebase this :)

assuming pcntl is an unix-only extension and they all support the siginfo_t type,
 we re using exclusively this more advanced api.
@devnexen devnexen force-pushed the pcntl_signal_simplification branch from b134b17 to bcffcc8 Compare April 6, 2026 19:38
@Girgias Girgias requested a review from petk April 6, 2026 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants