Skip to content

POSIX: add changelog entries for PHP 8.5 changes#5534

Open
lacatoire wants to merge 2 commits into
php:masterfrom
lacatoire:changelog/posix-8.5
Open

POSIX: add changelog entries for PHP 8.5 changes#5534
lacatoire wants to merge 2 commits into
php:masterfrom
lacatoire:changelog/posix-8.5

Conversation

@lacatoire
Copy link
Copy Markdown
Member

PHP 8.5 brings several POSIX changes already documented in
`appendices/migration85` but missing from the per-function changelogs.

  • posix_kill: ValueError on out-of-range process_id
  • posix_setpgid: ValueError on out-of-range process_id/process_group_id
  • posix_setrlimit: ValueError on invalid hard_limit/soft_limit
  • posix_ttyname: last_error EBADF on invalid file_descriptor
  • posix_isatty: E_WARNING on invalid file_descriptor
  • posix_fpathconf: last_error EBADF + E_WARNING on invalid file_descriptor

Same spirit as #5528.

Documented in appendices/migration85, missing from per-function changelogs:

- posix_kill: ValueError on out-of-range process_id
- posix_setpgid: ValueError on out-of-range process_id/process_group_id
- posix_setrlimit: ValueError on invalid hard_limit/soft_limit
- posix_ttyname: last_error EBADF on invalid file_descriptor
- posix_isatty: E_WARNING on invalid file_descriptor
- posix_fpathconf: last_error EBADF + E_WARNING on invalid file_descriptor
@lacatoire
Copy link
Copy Markdown
Member Author

same as #5528 @jordikroon

@jordikroon
Copy link
Copy Markdown
Member

I will get to this in a moment, but please be patient as I am also looking after a few older MR's.

Copy link
Copy Markdown
Member

@jordikroon jordikroon left a comment

Choose a reason for hiding this comment

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

The following 8.5 change also needs to be documented.

posix_setpgid throws a ValueError when the process_id or
the process_group_id is lower than zero or greater than
what the platform supports.

Comment thread reference/posix/functions/posix-kill.xml Outdated
Comment thread reference/posix/functions/posix-setpgid.xml Outdated
Aligns posix-kill / posix-setpgid changelog wording with posix-setrlimit
in the same PR and with the surrounding doc-en convention.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants