Bug#1061469: phpX.Y-mysql does not provide php-mysql virtual package

[Jeremy Davis]

Dear Alexandre (adminer maintainer),




On 25/1/24 16:05, Ondřej Surý wrote:

As per Ondřej's note below, with regards to MySQL support, from a glance through the source code (adminer/drivers/mysql.inc.php) it seems that adminer should depend on php-mysql (real metapackage) OR php-mysqli (virtual package provided by real phpX.Y-mysql package - i.e. currently php8.2-mysql).

The extension package provide names of the extensions actually provided by the 
said package. “mysql” extension has been removed from PHP a quite a while ago 
and is not provided by php8.2-mysql package. (Similar situation can be found in 
php8.2-xml package.)

This needs to be fixed in the adminer, so it actually depends on an extension 
it actually uses.

Ondrej
--
Ondřej Surý (He/Him)

Regards,
Jeremy


diff --git a/debian/control b/debian/control
index d6f56f0..58d1d56 100644
--- a/debian/control
+++ b/debian/control
@@ -14,7 +14,7 @@ Package: adminer
 Architecture: all
 Depends:
  libapache2-mod-php | php-cgi | php-fpm | php,
- php-mysql | php-sqlite3 | php-pgsql,
+ php-mysql | php-mysqli | php-sqlite3 | php-pgsql,
  ${misc:Depends},
 Recommends:
  php-cli,

Please find a patch attached to resolve this issue - #1061469. Hopefully I got it right? If you would prefer me to open a merge request on Salsa, please let me know and I'll do that instead (I don't have a salsa account but AFAIK it's relatively straight forward to get one). PS Actually, I've just noticed #1054574 & #1055329 - IMO it's probably not worth applying this and just move on with AdminerEvo!?

OpenPGP_signature.asc
Description: OpenPGP digital signature