mesonbuild/meson

`meson setup` output ignores `b_colorout`

Open

#13.708 aberto em 23 de set. de 2024

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Python (1.465 forks)batch import
enhancementgood first issue

Métricas do repositório

Stars
 (4.978 stars)
Métricas de merge de PR
 (Mesclagem média 30d 19h) (63 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Investigue o mecanismo de registro em `mesonbuild/mlog.py` em torno das linhas 395-398. O problema é que `meson setup` não respeita a flag ` Db colorout=always` quando a saída padrão não é um TTY. Para corrigir, adicione uma verificação para habilitar a saída colorida com base na opção `b colorout`, mesmo quando não for um TTY. Consulte o código vinculado e considere consultar os comentários dos mantenedores, se houver.
Pilha de tecnologia
python
Domain
build systemcli
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Basic Python knowledgeFamiliarity with Meson build system
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80