mesonbuild/meson

`meson setup` output ignores `b_colorout`

Open

#13 708 ouverte le 23 sept. 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Python (1 465 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (4 978 stars)
Métriques de merge PR
 (Merge moyen 30j 19h) (63 PRs mergées en 30 j)

Description

Describe the bug

meson setup -Db_colorout=always does not cause meson setup to produce color output when stdout is not a TTY. It appears that the logging code does not check the flag:

https://github.com/mesonbuild/meson/blob/dbad301c8b039c4469f8727924baf250421d64ac/mesonbuild/mlog.py#L395-L398

To Reproduce

$ cat > meson.build <<EOF
project('c', 'c')
summary('test', true, bool_yn: true)
EOF
$ meson setup build -Db_colorout=always | cat

Expected behavior

The meson setup log includes bold and green text.

system parameters

  • Is this a cross build or just a plain native build (for the same computer)? Native
  • what operating system (e.g. MacOS Catalina, Windows 10, CentOS 8.0, Ubuntu 18.04, etc.) Fedora 40
  • what Python version are you using e.g. 3.8.0 3.12.5
  • what meson --version 1.5.2
  • what ninja --version if it's a Ninja build n/a

Guide contributeur

Direction de recherche
Examinez le mécanisme de journalisation dans `mesonbuild/mlog.py` autour des lignes 395-398. Le problème est que `meson setup` ne respecte pas l'option ` Db colorout=always` lorsque la sortie standard n'est pas un TTY. Pour corriger, ajoutez une vérification pour activer la sortie couleur en fonction de l'option `b colorout`, même lorsqu'il ne s'agit pas d'un TTY. Reportez vous au code lié et envisagez de consulter les commentaires des mainteneurs, s'il y en a.
Stack technique
python
Domaine
build systemcli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Basic Python knowledgeFamiliarity with Meson build system
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80