Skip to content

Set SA_ONSTACK too when zend signals are disabled and in pcntl - #9758

Closed
dunglas wants to merge 1 commit into
php:masterfrom
dunglas:fix/sa-onstack-no-signals
Closed

Set SA_ONSTACK too when zend signals are disabled and in pcntl#9758
dunglas wants to merge 1 commit into
php:masterfrom
dunglas:fix/sa-onstack-no-signals

Conversation

@dunglas

Copy link
Copy Markdown
Member

Similar to #9597, but when --disable-zend-signals is passed (currently necessary when embedding PHP in Go because of #9649), and for pnctl.

@dunglas
dunglasforce-pushed the fix/sa-onstack-no-signals branch from 04d9d21 to c0df175CompareOctober 16, 2022 10:24

@devnexendevnexen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LCTM

Sign up for freeto 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.

2 participants

@dunglas@devnexen