Skip to content

Remplacement de mysqlcheck par mariadbcheck si Debian13#3358

Closed
ktn001 wants to merge 2 commits into
jeedom:developfrom
ktn001:develop
Closed

Remplacement de mysqlcheck par mariadbcheck si Debian13#3358
ktn001 wants to merge 2 commits into
jeedom:developfrom
ktn001:develop

Conversation

@ktn001
Copy link
Copy Markdown
Contributor

@ktn001 ktn001 commented May 24, 2026

On continue à utiliser "mysqlcheck" si "mariadbcheck" n'existe pas

Description

La commande "mysqlcheck" a été remplacée par "mariadbchek" dans la version de mariadb livrée avec Debian 13.

Suggested changelog entry

Related issues/external references

Fixes #

Types of changes

  • Bug fix (non-breaking change which fixes)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
    • This change is only breaking for integrators, not for external standards or end-users.
  • Documentation improvement

PR checklist

ktn001 added 2 commits May 24, 2026 11:17
Un petit pas vers Debian 13

On continue d'utiliser "mysqlcheck" si "mariadbcheck" n'existe pas
On continue à utiliser "mysqlcheck" si "mariadbcheck" n'existe pas
@Salvialf
Copy link
Copy Markdown
Contributor

Salvialf commented May 24, 2026

Bonjour @ktn001,

Merci d'avoir identifié et remonté ce changement de commande à prévoir pour Debian 13.

La branche dédiée à la compatibilité Debian 13 est prévue pour ce genre de cas, pourrais-tu soumettre cette PR sur fix/debian13-compatibility plutôt que develop s'il te plaît ?

@Mips2648 Mips2648 marked this pull request as draft May 25, 2026 07:30
@Mips2648 Mips2648 requested a review from Salvialf May 25, 2026 07:31
@ktn001
Copy link
Copy Markdown
Contributor Author

ktn001 commented May 25, 2026

La dernière fois que j'ai fait un PR, j'avais suivi la doc et on m'a demandé de la refaire car la branche utilisée ne devait plus l'être.

Cette fois, je ne me suis pas basé sur la doc mais sur l'indication qui m'avait été donnée et il faut à nouveau que je refasse ce que j'ai fait parce je ne suis à nouveau pas dans la branche voulue.

Je revendrai lorsque vous aurez une doc à jour à laquelle on peut se fier.

@Salvialf
Copy link
Copy Markdown
Contributor

Salvialf commented May 25, 2026

La branche par défaut est bien develop sauf qu'ici une branche dédiée à cette fonctionnalité existe déjà, il est donc normal et pas trop compliqué d'effectuer une PR dessus.

Je vais reprendre ta PR manuellement sur la bonne branche.

Edit: c'est commit sur la bonne branche, merci encore pour ta contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants