From: Daniel Black <daniel@mariadb.org>
Date: Wed, 8 May 2024 01:33:17 +0000 (+1000)
Subject: Bug 1467006: explicit MariaDB support (#140) - version detection fix (#168)
X-Git-Tag: bugzilla-5.2~23
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d1acd81eb59908fc36a54d3874bc1faeec125d59;p=thirdparty%2Fbugzilla.git

Bug 1467006: explicit MariaDB support (#140) - version detection fix (#168)

Small post-fix on 35a060cbd9d880541ad7e6c02710c0c3647a3b19 as MariaDB-11.0 removed the 5.5.5- replication prefix.

The reliable way to have MariaDB detected is looking for "-MariaDB" in the version string. Distro packages may add a suffix so an unanchored match is pretty reliable.
---

diff --git a/Bugzilla/DB.pm b/Bugzilla/DB.pm
index f67eb8da1..9f30cac25 100644
--- a/Bugzilla/DB.pm
+++ b/Bugzilla/DB.pm
@@ -237,12 +237,13 @@ sub bz_check_server_version {
 
   my $sql_vers = $self->bz_server_version;
   if (((lc($db->{name}) eq 'mysql') || (lc($db->{name}) eq "mariadb"))
-    && ($sql_vers =~ s/^5\.5\.5-//)) {
-    # Version 5.5.5 of MySQL never existed. MariaDB >= 10 always puts '5.5.5-'
+    && ($sql_vers =~ s/^5\.5\.5-// || $sql_vers =~ /-MariaDB/)) {
+    # Version 5.5.5 of MySQL never existed. MariaDB = 10 always puts '5.5.5-'
     # at the front of its version string to get around a limitation in the
     # replication protocol it shares with MySQL.  So if the version starts with
     # '5.5.5-' then we can assume this is MariaDB and the real version number
-    # will immediately follow that.
+    # will immediately follow that.  This was removed in MariaDB-11.0.  The
+    # version should always contain "MariaDB" if it is indeed MariaDB.
     $db = DB_MODULE->{'mariadb'};
   }
   my $sql_dontwant = exists $db->{db_blocklist} ? $db->{db_blocklist} : [];