[P2][ci] Faire échouer installcheck dès qu'un contrôle intermédiaire échoue #58

Closed
opened 2026-09-29 20:34:49 +00:00 by silex · 0 comments
Owner

Gravité et périmètre

P2 / medium. Makefile et validation CI de l'installation, commit 65442d2.

Constat

La recette Makefile:112 et suivantes enchaîne installation, assertions et désinstallation avec des points-virgules sans fail-fast. Le statut du dernier test peut masquer un échec antérieur, y compris l'impossibilité d'exécuter le binaire installé. La CI invoque cette cible et hérite donc de ce faux vert.

Reproduction

Dans une copie temporaire : make installcheck MAKE=true. Les installations/désinstallations deviennent des no-op, plusieurs assertions échouent, l'exécution du binaire installé indique qu'il n'existe pas, mais la cible termine avec le code 0. Aucun fichier système installé.

Impact

Une installation cassée peut être présentée comme valide. Ne bloque pas directement la dictée, mais invalide une garantie importante du contrôle de livraison.

Correction et acceptation

Rendre la recette fail-fast en gardant le trap de nettoyage. Ajouter un contrôle de régression ou une injection reproductible prouvant que l'échec d'une installation, d'une assertion intermédiaire ou de l'exécution du binaire produit un statut non nul. Vérifier aussi le chemin nominal, les sentinelles étrangères et le nettoyage du DESTDIR temporaire.

## Gravité et périmètre P2 / medium. Makefile et validation CI de l'installation, commit `65442d2`. ## Constat La recette `Makefile:112` et suivantes enchaîne installation, assertions et désinstallation avec des points-virgules sans fail-fast. Le statut du dernier test peut masquer un échec antérieur, y compris l'impossibilité d'exécuter le binaire installé. La CI invoque cette cible et hérite donc de ce faux vert. ## Reproduction Dans une copie temporaire : `make installcheck MAKE=true`. Les installations/désinstallations deviennent des no-op, plusieurs assertions échouent, l'exécution du binaire installé indique qu'il n'existe pas, mais la cible termine avec le code 0. Aucun fichier système installé. ## Impact Une installation cassée peut être présentée comme valide. Ne bloque pas directement la dictée, mais invalide une garantie importante du contrôle de livraison. ## Correction et acceptation Rendre la recette fail-fast en gardant le trap de nettoyage. Ajouter un contrôle de régression ou une injection reproductible prouvant que l'échec d'une installation, d'une assertion intermédiaire ou de l'exécution du binaire produit un statut non nul. Vérifier aussi le chemin nominal, les sentinelles étrangères et le nettoyage du DESTDIR temporaire.
silex closed this issue 2026-09-29 21:23:11 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
2027a/murmur#58
No description provided.