mesonbuild/meson

`meson setup` output ignores `b_colorout`

Open

#13,708 opened on 2024年9月23日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)Python (4,978 stars) (1,465 forks)batch import
enhancementgood first issue

説明

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

コントリビューターガイド

技術スタック
python
領域
build systemcli
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Basic Python knowledgeFamiliarity with Meson build system
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
80
調査方針
Investigate the logging mechanism in `mesonbuild/mlog.py` around line 395-398. The issue is that `meson setup` does not respect the ` Db colorout=always` flag when stdout is not a TTY. To fix, add a check to enable color output based on the `b colorout` option, even when not a TTY. Refer to the linked code and consider consulting maintainer comments if any.