Skip to content

ext/standard: Use posix_spawn_file_actions_addchdir when available#21553

Open
NattyNarwhal wants to merge 1 commit intophp:masterfrom
NattyNarwhal:dont-use-addchdir-np
Open

ext/standard: Use posix_spawn_file_actions_addchdir when available#21553
NattyNarwhal wants to merge 1 commit intophp:masterfrom
NattyNarwhal:dont-use-addchdir-np

Conversation

@NattyNarwhal
Copy link
Copy Markdown
Member

posix_spawn_file_actions_addchdir is part of POSIX now, so some OSes (macOS at least) have started to deprecated the _np variant. Some support both names (Solaris, NetBSD), others don't yet (FreeBSD).

Use the non-np variant when possible to avoid the deprecation warning on macOS and other platforms in the future.

Fixes GH-21552

posix_spawn_file_actions_addchdir is part of POSIX now, so some OSes
(macOS at least) have started to deprecated the _np variant. Some
support both names (Solaris, NetBSD), others don't yet (FreeBSD).

Use the non-np variant when possible to avoid the deprecation warning on
macOS and other platforms in the future.

Fixes phpGH-21552
Copy link
Copy Markdown
Member

@iluuu1994 iluuu1994 left a comment

Choose a reason for hiding this comment

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

LGTM

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.

posix_spawn_file_actions_addchdir_np is deprecated in macOS 26

2 participants