[Concept,00/11] buildman: Pylint cleanups

Message ID	20260104200844.481633-1-sjg@u-boot.org
Headers	From: Simon Glass <sjg@u-boot.org> To: U-Boot Concept <concept@u-boot.org> Date: Sun, 4 Jan 2026 13:08:29 -0700 Message-ID: <20260104200844.481633-1-sjg@u-boot.org> MIME-Version: 1.0 Message-ID-Hash: WZLIUADO44H5XGYOLU6Y4ROYXPFHMJPS CC: Heinrich Schuchardt <xypron.glpk@gmx.de>, Simon Glass <simon.glass@canonical.com> Precedence: list Subject: [Concept] [PATCH 00/11] buildman: Pylint cleanups Archived-At: <https://lists.u-boot.org/archives/list/concept@u-boot.org/message/WZLIUADO44H5XGYOLU6Y4ROYXPFHMJPS/> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit
Series	buildman: Pylint cleanups \| [Concept,00/11] buildman: Pylint cleanups [Concept,01/11] buildman: Fix pylint warnings in main.py [Concept,02/11] claude: Drop mention of --no-ext-diff [Concept,03/11] buildman: Fix pylint warnings in bsettings.py [Concept,04/11] buildman: Use tools.read_file/write_file in test_bsettings.py [Concept,05/11] buildman: Rename toolchain methods to snake_case [Concept,06/11] buildman: Fix docstrings in toolchain.py [Concept,07/11] buildman: Make MyHTMLParser.re_arch private [Concept,08/11] buildman: Document all Toolchain public members [Concept,09/11] buildman: Use f-strings in toolchain.py [Concept,10/11] buildman: Fix remaining pylint warnings in toolchain.py [Concept,11/11] buildman: Fix pylint warnings in test_boards.py

Message ID

20260104200844.481633-1-sjg@u-boot.org

Headers

From: Simon Glass <sjg@u-boot.org>
To: U-Boot Concept <concept@u-boot.org>
Date: Sun,  4 Jan 2026 13:08:29 -0700
Message-ID: <20260104200844.481633-1-sjg@u-boot.org>
MIME-Version: 1.0
Message-ID-Hash: WZLIUADO44H5XGYOLU6Y4ROYXPFHMJPS
CC: Heinrich Schuchardt <xypron.glpk@gmx.de>,
 Simon Glass <simon.glass@canonical.com>
Precedence: list
Subject: [Concept] [PATCH 00/11] buildman: Pylint cleanups
Archived-At: 
 <https://lists.u-boot.org/archives/list/concept@u-boot.org/message/WZLIUADO44H5XGYOLU6Y4ROYXPFHMJPS/>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

Series

buildman: Pylint cleanups |

Message

Simon Glass Jan. 4, 2026, 8:08 p.m. UTC

  From: Simon Glass <simon.glass@canonical.com>

This series cleans up pylint warnings across several buildman modules,
bringing them all to 10.00/10:

- main.py: Remove dead imports, fix docstrings, fix control flow
- bsettings.py: Add docstrings, fix exception handling, use f-strings
- test_bsettings.py: Use tools.read_file/write_file helpers
- toolchain.py: Rename CamelCase methods to snake_case, add docstrings,
  document public members, use f-strings, fix various warnings
- test_boards.py: Fix long lines, unused variables

The toolchain.py changes are split into multiple commits for easier
review, as the file required significant updates.


Simon Glass (11):
  buildman: Fix pylint warnings in main.py
  claude: Drop mention of --no-ext-diff
  buildman: Fix pylint warnings in bsettings.py
  buildman: Use tools.read_file/write_file in test_bsettings.py
  buildman: Rename toolchain methods to snake_case
  buildman: Fix docstrings in toolchain.py
  buildman: Make MyHTMLParser.re_arch private
  buildman: Document all Toolchain public members
  buildman: Use f-strings in toolchain.py
  buildman: Fix remaining pylint warnings in toolchain.py
  buildman: Fix pylint warnings in test_boards.py

 CLAUDE.md                        |   1 -
 tools/buildman/bsettings.py      |  60 +++---
 tools/buildman/builder.py        |   2 +-
 tools/buildman/builderthread.py  |   6 +-
 tools/buildman/control.py        |  16 +-
 tools/buildman/func_test.py      |   6 +-
 tools/buildman/main.py           |  30 ++-
 tools/buildman/test.py           |  90 ++++----
 tools/buildman/test_boards.py    |  72 ++++---
 tools/buildman/test_bsettings.py |  16 +-
 tools/buildman/toolchain.py      | 350 +++++++++++++++++--------------
 tools/qconfig.py                 |   8 +-
 12 files changed, 346 insertions(+), 311 deletions(-)