Adapt install scripts and Core to Debian 13 by TiTidom-RC · Pull Request #3176 · jeedom/core

TiTidom-RC · 2026-01-02T10:46:55Z

Description

Adapt the install scripts and some core functions to ensure Jeedom can be installed on Debian 13, while maintaining backward compatibility with Debian 11 and 12.

Updated the Docker installation files and scripts too.

Suggested changelog entry

All changes are detailed in the _trixie_migrationcheck.md and _trixie_retrocheck.md files. These files are located at the root of the repository.

In summary, here are the key changes :

Replaced Debian Packages :

ntp → chrony
ntpdate → removed (included in chrony)
locate → plocate (modern version with alternative)
espeak + mbrola → espeak-ng
libcurl3-gnutls → libcurl4 (for Debian 13+)

Removed Debian packages :

telnet
apt-transport-https (inluded by default since Debian 10)
cutycapt
dos2unix (replaced by "sed")
libav-tools (replaced by "ffmpeg")
software-properties-common (Ubuntu package, not available in Debian)

Migrate to PHP-FPM :

proxy_fcgi
setenvif

Update MariaDB parameters :

Removed parameters (deprecated) :

query_cache_type, query_cache_size, query_cache_limit, query_cache_min_res_unit (query cache no more available)
innodb_large_prefix (active by default)

Support for custom fork repository (for the install script) :

Add "-r" parameter (R like repository) in install's script :

sudo ./install.sh -r owner/repo -v branche

Updated all references from 'apt' to 'apt-get', adding options to handle conflicts and interactions with shell.

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

I have checked there is no other PR open for the same change.
I have read the [La ligne directrice pour contribuer à ce projet / Contribution guidelines for this project).
I grant the project the right to include and distribute the code under the GNU.
I have added tests to cover my changes.
I have verified that the code complies with the projects coding standards.
[Required for new sniffs] I have added MD documentation for the sniff.

Update code to be able to install on Debian 13

Avoid error messages (operation not permitted on chmod) after install

zoic21

Beaucoup de changement que je ne comprends pas vraiment

zoic21 · 2026-01-30T10:13:52Z

 		if (!file_exists(__DIR__ . '/../../cache.tar.gz')) {
-			$cmd = 'mkdir ' . $cache_dir . ';';
-			$cmd .= 'chmod -R 777 ' . $cache_dir . ';';
+			$cmd = system::getCmdSudo() . 'mkdir -p ' . $cache_dir . ';';


Normalement tout le cache est sous www-data avec ce changement il passe en root ce qui n'est pas top

Effectivement, le sudo mkdir va créer le dossier en root, mais aux lignes 402 et 403 (juste en dessous) je gère le owner du dossier en le réattribuant à "www-data" (user et groupe d'ailleurs).

J'ai utilisé le "sudo" par "sécurité", au cas où il y aurait des problèmes de droits existants sur le répertoire parent, et j'ai surtout utilisé le paramètre "-p" pour créer l'arborescence si elle n'existe pas.

Par les lignes d'en dessous, je m'assure que le répertoire appartient bien à "www-data" et ainsi ne pas gêner Jeedom dans son fonctionnement.

zoic21 · 2026-01-30T10:14:44Z

+echo "ServerName localhost" >> /etc/apache2/apache2.conf
+
+echo 'Start PHP-FPM'
+PHP_VERSION=$(php -r "echo PHP_MAJOR_VERSION.'.'.PHP_MINOR_VERSION;")


Je veux bien plus de détails la dessus

oui bien sûr.

J'ai basculé le script d'install pour utiliser le module PHP "FPM" couplé à "mod_proxy_fcgi" côté Apache. A l'install, le paquet linux "fpm" n'a pas d'appellation / alias "générique" sous Linux, mais est du style "php8.3-fpm" (donc contient la version php dans son nom). Pour que le script d'init reste valide pour n'importe quelle version qui sera installée maintenant mais également à l'avenir, je le passe en mode dynamique, en calculant la version PHP qui est installée, et en appelant donc le service "fpm" correspondant (ligne 157).

Si la question était pour le fait de démarrer le service, le module fpm ne démarre pas tout seul après son install, et dans un docker, il n'y a pas systématiquement de "systemd", donc je force le démarrage du service (ligne 157) avec son nom complet (incluant donc la version de PHP).

Pourquoi basculer vers PHP FPM dans ce PR?
Si le but est d'utiliser FPM au lieu de CGI juste pour raison de perf, indépendamment de trixie, alors il ne faudrait pas inclure ce changement dans cette branche

Je comprend le point de vue.

L'intégration de Debian 13 (et je précise comme dit par ailleurs : le script n'est pas seulement compatible Debian 13, mais également Debian 12 et même Debian 11), nécessitait de toute façon de revoir tout le script d'installation, donc plutôt que de conserver la stack CGI (vieillissante et plus le standard du marché), et que le passage à FPM impliquait de revoir ce script d'install (et comme ici la partie docker), pour aller au bout de l'exercice et pouvoir tester de bout en bout, j'ai intégré la partie passage à FPM dans ce PR.

Un changement d'OS devrait être l'occasion de moderniser l'ensemble et de revoir toute la stack 👍 (c'est dans cette optique que j'ai intégré l'ensemble de ces changements, celui-ci et tous les autres, tout en garantissant la rétrocompatibilité).

Hello, ça m'intéresse ;)

Par contre, non il vaudrait mieux séparer et faire une 2e PR pour la "migration" CGI vers FPM si besoin, il n'y a pas besoin de la faire (les 2 sont supporté par debian 11 / 12 / 13 donc ce n'est pas une nécessité techniquement), et la séparation des 2 permet de mieux isoler les problèmes qu'on ne manquera pas de découvrir... 😄

Mais ça m'intéresse parce que j'utilise l'image Docker officielle php pour l'image Jeedom, et donc celle avec apache, mais j'ai essayé d'utiliser celle avec FPM mais jamais réussi à le configurer. FPM implique d'utiliser nginx au lieu de apache (du moins dans tous les exemples que j'ai trouvé). Si tu fais une image Docker avec Jeedom FPM, il faut bien une autre image avec le serveur http ?

zoic21 · 2026-01-30T10:15:30Z

 elif [[ "${GITHUB_REF_NAME}" == "beta" ]]; then
  JEEDOM_TAGS="${REPO}/jeedom:beta"; # ${REPO}/jeedom:$JEEDOM_SHORT_VERSION";
  GITHUB_BRANCH=${GITHUB_REF_NAME};
+elif [[ "${GITHUB_REF_NAME}" == "trixie" ]]; then


Pour moi on devrait pas avoir une branche trixie....

dans le principe, je suis tout à fait d'accord avec toi, mais il me fallait bien le moyen de tester le script d'install de bout en bout, et la branche "trixie" était son nom actuel 👍

De la même manière, j'imagine qu'à terme, les lignes 19 et 20 (parlant de la branche alpha) devraient disparaitre également, non ?

ce fichier init_workflow.sh est spécifique au workflow github et j'ai proposé de le déplacer dans .github/script dans une autre PR #3172
Il faudra revoir le nom des branches j'attends toujours de savoir comment...
https://community.jeedom.com/t/2026-pr-a-faire-sur-branche-alpha-ou-beta/145747/15

NebzHB · 2026-04-16T12:27:01Z

 						} else {
 							if ($_foreground) {
-								echo shell_exec(self::getCmdSudo() . ' apt update;' . self::getCmdSudo() . ' apt-get install -y python3 python3-pip python3-dev python3-venv');
+								echo shell_exec(self::getCmdSudo() . ' apt-get update;' . self::getCmdSudo() . ' apt-get install -y python3 python3-pip python3-dev python3-venv');


Pourquoi ne pas retirer l'interactive et garder les anciennes conf aussi ?

NebzHB · 2026-04-16T12:27:13Z

 								echo shell_exec(self::getCmdSudo() . self::getCmdPython3($_plugin) . ' -m pip install --upgrade pip wheel');
 							} else {
-								$cmd .= self::getCmdSudo() . " apt update;\n" . self::getCmdSudo() . " apt-get install -y python3 python3-pip python3-dev python3-venv\n";
+								$cmd .= self::getCmdSudo() . " apt-get update;\n" . self::getCmdSudo() . " apt-get install -y python3 python3-pip python3-dev python3-venv\n";


NebzHB · 2026-04-16T12:34:55Z

 setTimeZone() {
-  [[ ${TZ} == $(</etc/timezone) ]] && return
+  # Check if timezone is already correctly configured via /etc/localtime
+  if [[ -L /etc/localtime ]] && [[ $(readlink /etc/localtime) == "/usr/share/zoneinfo/${TZ}" ]]; then


refactor qui a l'air de venir de l'IA, on est certain que zoneinfo est à cet endroit sur RPI ou debian ou autre ?

NebzHB · 2026-04-16T12:36:09Z

 	JEEDOM_INSTALL=0
 	rm -rf /root/install.sh
-	wget https://raw.githubusercontent.com/jeedom/core/${VERSION}/install/install.sh -O /root/install.sh
+	wget https://raw.githubusercontent.com/${GITHUB_REPO}/${VERSION}/install/install.sh -O /root/install.sh


utilité ? ca sera toujours jeedom/core non ?

TiTidom-RC added 30 commits January 2, 2026 11:27

Compatibility with Trixie

212ab3c

Update code to be able to install on Debian 13

update install script

e370e00

update install script

bc44601

Update install.sh

e897d01

update install script + doc trixie

8147877

update install script

89b7e11

Update script install_nodejs

5c561ae

update nodejs install script

f819608

update trixie migrate doc

5baf50c

Update install_nodejs.sh

110d3ba

Update install_nodejs.sh

af36ea8

Update install script

bd7e970

Update system.class.php

f26e334

change apt to apt-get

34f4b0b

update _trixie MD

8c2dd52

update interactions in install script

56cc81f

update _trixie MD

c1ea0d6

update shell interactions in install script

39f9585

update install_openssl.sh

66d7515

update wget options in install scripts

c3737dc

update wget options to redirect logs to stdout

9404ebc

Merge remote-tracking branch 'upstream/master' into trixie

680428e

Merge remote-tracking branch 'upstream/4.5.2' into trixie

19e18b9

update _trixie docs

5e74a14

add apt clean in install script

ed8316e

Update Dockerfile for Debian 13

ccf252b

add trixie tag for docker

5d05077

include GITHUB_REPO var in init.sh

5084f90

Update _trixie_migrationcheck.md

f366727

Update Dockerfile

5f6c888

TiTidom-RC added 12 commits January 2, 2026 11:29

Update Dockerfile

d2873ef

Update init.sh

84ec09c

typo in init_workflow.sh

161393c

Update _trixie_migrationcheck.md

1133845

Update _trixie_migrationcheck.md

48fc559

Update _trixie_migrationcheck.md

6d37ed4

Update _trixie_migrationcheck.md

17906a2

Update cache restore function

2a3352a

Avoid error messages (operation not permitted on chmod) after install

Update _trixie_migrationcheck.md

6dc357d

Update init.sh

9299e48

Merge branch 'trixie' into trixie

b8bf6f1

add mariadb-client-compat to the install scripts

3708eec

zoic21 requested changes Jan 30, 2026

View reviewed changes

zoic21 added the Need review label Jan 30, 2026

NebzHB reviewed Apr 16, 2026

View reviewed changes

Conversation

TiTidom-RC commented Jan 2, 2026

Description

Suggested changelog entry

Related issues/external references

Types of changes

PR checklist

Uh oh!

zoic21 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants