Skip to content

PHP 8.1 Compatibility Fixes #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: trunk
Choose a base branch
from

Conversation

kirmorozov
Copy link
Contributor

PR fixes build process for PHP. 8.1,
Breaking changes from PHP source:
removed MYSQLND_SZ_T_SPEC
php/php-src@b80767e
Change of mysqlnd_error_info_init
php/php-src@7088cff

@temuri416
Copy link

Thanks for the fix, @kirmorozov. Any idea when this going to be merged?

@temuri416
Copy link

btw there's a syntax error - double closing parenthesis:

11fc863#diff-7561e12b39e0462dcc1c6c7ddcd6ff63a7f63e82e197a0a718a05fe0407a4f4cR779

Change of mysqlnd_error_info_init
php/php-src@7088cff
@cmb69
Copy link
Member

cmb69 commented Jan 18, 2022

I can confirm that these changes are required for PHP 8.1 compatibility.

@rsomla1
Copy link
Contributor

rsomla1 commented May 11, 2022

Thank you for the fix @kirmorozov. We will include it in the next release. Sorry it took so long...

@temuri416
Copy link

@rsomla1 Any ETA on the patch? Thank you!

@rsomla1
Copy link
Contributor

rsomla1 commented Jun 27, 2022

Hi, It will be part of 8.0.30 release due in a month or so.

@SugarD-x
Copy link

SugarD-x commented Oct 6, 2022

Sorry to be a pest, but any news on this patch and/or that version being released? It's been quite awhile and neither has come to light yet.

@rsomla1
Copy link
Contributor

rsomla1 commented Oct 6, 2022

Hi, it was included in 8.0.30 and is part of release/8.0.30 branch: e7b03c1

But, for some reason, it is not in trunk. I will check what is going on there.

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.

5 participants