Skip to content
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

Compile fails with MySQL 8.3.0 due to MySQL removing a deprecated function DBD::mysql uses #411

Closed
tzs opened this issue Feb 7, 2024 · 4 comments · Fixed by #415
Closed
Assignees
Labels
Milestone

Comments

@tzs
Copy link

tzs commented Feb 7, 2024

DBD::mysql version

DBD-mysql-5.003

MySQL client version

8.3.0

Server version

No response

Operating system version

MacOS Sonoma 14.2.1

What happened?

Build fails with this error:

dbdimp.c:2796:14: error: call to undeclared function 'mysql_list_fields'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
*result= mysql_list_fields(svsock, table, NULL);
. ^
It looks like mysql_list_fields was removed in MySQL 8.3.0, which was released 3 weeks ago.

Other information

No response

@tzs tzs added the bug label Feb 7, 2024
@DrHyde
Copy link

DrHyde commented Feb 26, 2024

After comparing the relevant bit of code to that in DBD::MariaDB, I wonder if the call to mysql_list_fields (and its surroundings) can just be got rid of. It seems to be in some error checking that is then repeated a few moments later when the query is executed using mysql_send_query or mysql_real_query.

@DrHyde
Copy link

DrHyde commented Feb 26, 2024

For those using Mac OS and homebrew to manage their mysql, I recommend pinning your mysql until this is fixed.

But if you're too late and it's only after a brew upgrade broke things that you ended up looking at this issue, then you can brew tap DrHyde/homebrew-DrHyde and brew install mysql-82 to temporarily downgrade.

@dveeden dveeden added this to the 5.004 milestone Feb 27, 2024
@dveeden
Copy link
Collaborator

dveeden commented Mar 12, 2024

Looks like mysql_reload() now also starts to cause issues: #414 (comment)

@dveeden
Copy link
Collaborator

dveeden commented Mar 12, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants