Skip to content

Revert deprecation of __sleep and __wakeup#19966

Merged
bukka merged 3 commits intophp:PHP-8.5from
nicolas-grekas:revert-sleep-wakeup-deprec
Oct 6, 2025
Merged

Revert deprecation of __sleep and __wakeup#19966
bukka merged 3 commits intophp:PHP-8.5from
nicolas-grekas:revert-sleep-wakeup-deprec

Conversation

@nicolas-grekas
Copy link
Contributor

@nicolas-grekas nicolas-grekas commented Sep 25, 2025 •
edited
Loading

Without precluding the result of the vote at https://wiki.php.net/rfc/soft-deprecate-sleep-wakeup

I'm trying to help PHP 8.5 RMs get ready for RC2, as advised in this message:
https://externals.io/message/128643#128659

This PR reverts the commits on the topic, related to PRs #19682 and #19435
It also reverts #19827

Reviews welcome to ensure this can be merged on Oct. 4th if the vote passes.

/cc @DanielEScherzer @edorian

@nicolas-grekas nicolas-grekas changed the base branch from master to PHP-8.5 September 25, 2025 18:23
@nicolas-grekas nicolas-grekas force-pushed the revert-sleep-wakeup-deprec branch 2 times, most recently from 30913e3 to 6f3c8f2 Compare September 25, 2025 18:27
@nicolas-grekas
Copy link
Contributor Author

nicolas-grekas commented Sep 25, 2025 •
edited
Loading

I'm wondering if it's worth reverting #19827 given that the affected classes already have __[un]serialize hooks since a few years now (since PHP 8.2).

TimWolla
TimWolla reviewed Sep 25, 2025
View reviewed changes
TimWolla
TimWolla reviewed Sep 25, 2025
View reviewed changes
@nicolas-grekas nicolas-grekas force-pushed the revert-sleep-wakeup-deprec branch from a7d54d1 to 0375921 Compare September 26, 2025 07:23
@nicolas-grekas
Copy link
Contributor Author

nicolas-grekas commented Sep 26, 2025 •
edited
Loading

Thanks for the review, I removed the commits that reverted NEWS updates and also removed the revert of #19827, which can and should be kept to me.
I also added a commit to update the NEWS/UPGRADING files to mention the soft-deprecation.

@nicolas-grekas nicolas-grekas force-pushed the revert-sleep-wakeup-deprec branch from 0375921 to ea69ff6 Compare September 26, 2025 07:26
TimWolla
TimWolla reviewed Sep 26, 2025
View reviewed changes
@TimWolla TimWolla added the RFC label Sep 26, 2025
@nicolas-grekas nicolas-grekas force-pushed the revert-sleep-wakeup-deprec branch from ea69ff6 to 32ffdd0 Compare September 26, 2025 07:50
@xabbuh xabbuh mentioned this pull request Sep 29, 2025
Merged
Girgias
Girgias reviewed Sep 30, 2025
View reviewed changes
Copy link
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

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

Minor comments, but LGTM otherwise.

@nicolas-grekas nicolas-grekas force-pushed the revert-sleep-wakeup-deprec branch from 32ffdd0 to 0b844d4 Compare September 30, 2025 13:56
ndossche
ndossche approved these changes Sep 30, 2025
View reviewed changes
Copy link
Member

@ndossche ndossche left a comment

Choose a reason for hiding this comment

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

Looks fine from a technical PoV

theodorejb
theodorejb reviewed Oct 1, 2025
View reviewed changes
edorian
edorian approved these changes Oct 2, 2025
View reviewed changes
Copy link
Member

@edorian edorian left a comment

Choose a reason for hiding this comment

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

On the, likely, condition that the RFC passes: 👍 RM wise.

Thank you for getting everything ready, making it easy to stay within the plan made to get this sorted cleanly. Much appreciated.

@nicolas-grekas nicolas-grekas force-pushed the revert-sleep-wakeup-deprec branch from 0b844d4 to e5ae1d8 Compare October 2, 2025 11:09
@nicolas-grekas nicolas-grekas force-pushed the revert-sleep-wakeup-deprec branch from e5ae1d8 to f58da91 Compare October 6, 2025 06:47
@bukka bukka merged commit fc35396 into php:PHP-8.5 Oct 6, 2025
9 checks passed
@DanielEScherzer
Copy link
Member

DanielEScherzer commented Oct 6, 2025

I was planning to review this today and then merge it, sorry for the delay - looks good to me and I guess has already been merged

@jnoordsij jnoordsij mentioned this pull request Oct 7, 2025
Closed
@xabbuh xabbuh mentioned this pull request Oct 7, 2025
Closed
@kubawerlos kubawerlos mentioned this pull request Oct 16, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TimWolla TimWolla TimWolla left review comments

@Girgias Girgias Girgias left review comments

@DanielEScherzer DanielEScherzer DanielEScherzer left review comments

@edorian edorian edorian approved these changes

@ndossche ndossche ndossche approved these changes

@bukka bukka Awaiting requested review from bukka bukka is a code owner

@kamil-tekiela kamil-tekiela Awaiting requested review from kamil-tekiela kamil-tekiela is a code owner

@SakiTakamachi SakiTakamachi Awaiting requested review from SakiTakamachi SakiTakamachi is a code owner

@derickr derickr Awaiting requested review from derickr derickr is a code owner

@kocsismate kocsismate Awaiting requested review from kocsismate

+1 more reviewer

@theodorejb theodorejb theodorejb left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Category: Engine Extension: date Extension: dom Extension: pdo_mysql Extension: phar Extension: session Extension: spl Extension: standard RFC

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@nicolas-grekas @DanielEScherzer @TimWolla @edorian @theodorejb @ndossche @Girgias @bukka