CVS commit: pkgsrc/databases/p5-DBD-mysql

["Greg Troxel" <gdt%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   gdt
Date:           Fri Sep 19 23:14:31 UTC 2025

Modified Files:
        pkgsrc/databases/p5-DBD-mysql: Makefile

Log Message:
databases/p5-DBD-mysql: Expand MariaDB comment

The situation is messier than it appears, as upstream's recommendation
to use p5-DBD_MariaDB cannot be straightforwardly used.  Explain more
and include upstream bug report URL.   Leave a hint that version 4
allowed MariaDB, for those wishing a workaround -- actually fixing
this appears very difficult.


To generate a diff of this commit:
cvs rdiff -u -r1.87 -r1.88 pkgsrc/databases/p5-DBD-mysql/Makefile

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/databases/p5-DBD-mysql/Makefile
diff -u pkgsrc/databases/p5-DBD-mysql/Makefile:1.87 pkgsrc/databases/p5-DBD-mysql/Makefile:1.88
--- pkgsrc/databases/p5-DBD-mysql/Makefile:1.87 Sun Aug 17 10:00:43 2025
+++ pkgsrc/databases/p5-DBD-mysql/Makefile      Fri Sep 19 23:14:31 2025
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.87 2025/08/17 10:00:43 wiz Exp $
+# $NetBSD: Makefile,v 1.88 2025/09/19 23:14:31 gdt Exp $
 
 DISTNAME=      DBD-mysql-5.013
 PKGNAME=       p5-${DISTNAME}
@@ -19,7 +19,18 @@ REPLACE_PERL=                lib/DBD/mysql.pm
 
 MAKE_PARAMS+=          --nocatchstderr
 
-# newer mysql versions should be fine too; for mariadb, use DBD::MariaDB
+# Version 5 of p5-DBD-mysql drops support for building against mariadb
+# client libraries.  (Version 4 allowed this.)
+# Upstream does not consider this a bug:
+#   https://github.com/perl5-dbi/DBD-mysql/issues/481
+# and says that users should use p5-DBD-MariaDB.  However, that is not
+# interchangeable, erroring on at least some legal settings with
+# DBD:mysql.
+
+# Newer mysql versions should be fine too.  This is not
+# ${MYSQL_VERSIONS_ALL} because that includes MariaDB, exposing that
+# some think MariaDB should be considered a mysql version, and some
+# think not.
 MYSQL_VERSIONS_ACCEPTED=       80
 
 .include "../../databases/p5-DBI/buildlink3.mk"