MySQL Bugs: #110808: Contribution: sql/memory: Fix build on musl

Bug #110808	Contribution: sql/memory: Fix build on musl
Submitted:	25 Apr 2023 21:48	Modified:	20 Dec 2023 11:31
Reporter:	OCA Admin (OCA)	Email Updates:
Status:	Closed	Impact on me:	None
Category:	MySQL Server: Compiling	Severity:	S3 (Non-critical)
Version:	8.0	OS:	Any
Assigned to:		CPU Architecture:	Any
Tags:	Contribution

Description:
This bug tracks a contribution by Sam James (Github user: thesamesam) as described in http://github.com/mysql/mysql-server/pull/455

How to repeat:
See description

Suggested fix:
See contribution code attached

Contribution submitted via Github - sql/memory: Fix build on musl (*) Contribution by Sam James (Github thesamesam, mysql-server/pull/455#issuecomment-1519788787): I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.

Contribution: git_patch_1306642895.txt (text/plain), 744 bytes.

Hello Sam James,

Thank you for the report and contribution.

regards,
Umesh

Also seen this:
https://bugs.launchpad.net/ubuntu/+source/mysql-8.0/+bug/2019203

which is a slightly different patch.

This fixes one of two issues that arise when building with musl.

The other issue is that res_ninit, res_search and res_nclose are used in libmysql/dns_srv.cc for DNS SRV lookups but those are not available with musl.

Duplicate bug: https://bugs.mysql.com/bug.php?id=112995

We have another patch for the res issue you mention too: https://gitweb.gentoo.org/repo/gentoo.git/tree/dev-db/mysql-connector-c/files/mysql-connec.... It's not perfect but I'm not really sure there's a great alternative to it.

Sam, could you submit that patch via a github PR or some other way that Oracle can verify that this is under the OCA so they can use it?

Sure, thanks for the quick reply. Lemme do it now before I forget.

Done as https://github.com/mysql/mysql-server/pull/505.

Documented fix as follows in the MySQL 8.0.36 and 8.4.0 changelogs:

    MySQL did not build correctly using musl libc.

    Our thanks to Sam James for the contribution.

Closed.