Georg Richter added a comment - 2024-12-12 10:06 rsilen : Do we really need LIBMYSQL_VERSION_ID beside MARIADB_PACKAGE_ID ? For compile time checking MARIADB_PACKAGE_ID could be used: #ifdef MARIADB_PACKAGE_ID #if MARIADB_PACKAGE_ID > 30400 #define HAVE_PARSEC_PLUGIN #endif #endif For runtime checking mysql_get_client_version should be used: if (mysql_get_client_version(mysql) < 30400) { set_error( "Parsec plugin is not supported in this version" ); return 1; }

Daniel Black added a comment - 2025-01-05 23:48 I looked at the inspircd implementation on https://github.com/inspircd/inspircd/blob/insp4/src/modules/extra/m_mysql.cpp#L342 which is effectively LIBMYSQL_VERSION_ID > 80000 a few time on recent MySQL added/changed features. The sentiment from https://github.com/inspircd/docs/issues/239#issuecomment-2272862199 [T]here have been some additions to MySQL that have not been replicated in MariaDB. I'm unsure as to whether they plan to make future changes which will make it even more incompatible and I'd like to avoid the situation we ended up in with LibreSSL where we documented support and they ended up breaking so much compatibility that we ended up with a mess of fragile compatibility code in v3 and had to drop support in v4. Inspird usages relate to CONC-729 (ssl) and CONC-728 mysql_real_connect_dns_srv Runtime checks aren't the solution to providing a MariaDB C/C and MySQL libmysql as it will compile fail on missing symbols. How ABI compatible is a drop in MariaDB C/C against a MySQL compiled source? On LIBMYSQL_VERSION_ID, I think the current versioning below MySQL is saving us from compile incompatibilities. There was some versions that used the server version and I think this is where troubles arised. Is this documented somewhere because I don't see it on the usual places https://mariadb.com/kb/en/mariadb-connectorc-types-and-definitions/#macros https://github.com/mariadb-corporation/mariadb-connector-c/wiki/ . Using MARIADB_PACKAGE_ID for supporting MariaDB features is definitely recommended. I don't see a way we can concretely map LIBMYSQL_VERSION_ID in a way that 100% matches MySQL's compatibility. MYSQL_VERSION__ID gets used in messages like https://github.com/inspircd/inspircd/blob/insp4/src/modules/extra/m_mysql.cpp#L479 so I see no reason why LIBMYSQL_VERSION_ID wouldn't be used this way elsewhere without any impack on MariaDB versioning. For SSL a patten of: #ifdef MYSQL_OPT_SSL_MODE unsigned int ssl = config->getBool( "ssl" ) ? SSL_MODE_REQUIRED : SSL_MODE_PREFERRED; mysql_options(connection, MYSQL_OPT_SSL_MODE, &ssl); #endif would avoid version checking. For mysql_real_connect_dns_srv I don't an easy way besides version checking. So the TLDR is LIBMYSQL_VERSION_ID (and MYSQL_VERSION_ID) can't reasonably follow the MySQL unless we pin at the lowest compatible 5.7 version and I don't think that's a good idea. Better documentation on how to achieve a cross compatible interface around multiple MariaDB C/C and MySQL libmysql versions would assist in our engagement with user communities.