From patchwork Tue Jan 6 22:03:18 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 90 Return-Path: X-Original-To: u-boot-concept@u-boot.org Delivered-To: u-boot-concept@u-boot.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767737058; bh=WF+B8BUb1UAD/9XvlxrKxoTmwhTWNOqgfhmTNRgliKQ=; h=From:To:Date:CC:Subject:List-Id:List-Archive:List-Help:List-Owner: List-Post:List-Subscribe:List-Unsubscribe:From; b=lSbwAWUVHOAsukh7AClkQgU5TbATWVCX9N+mQpHmFGOicj1v9OiL1UKFHKo4dtEoR YbzhUv1hThEZOoUkgqzuXCGQjO27Wf0R+DvOSU9vaThVEgc2z9dcZ0OoePTVKSTgQD EZP19TOQN9owCGVBdDW5G1wfaATVOONpRXwrU8o8l3IxEZPvIl324je2j9PlW5TY93 /O8depiyi+A5ZbRc0PUhZZlZCdxBbcy3i74kXP+1K4eQVfsAV71lGd1EuFtnzXDFYI WzMLZLMBMdRKL53LLC5Vq5j3Lt8bCdgmcKYddU+yp25s6ghfmmknt46dMk4uPBU/hk 7st0MCsXxUHKg== Received: from localhost (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id DE7EA69157 for ; Tue, 6 Jan 2026 15:04:18 -0700 (MST) X-Virus-Scanned: Debian amavis at Received: from mail.u-boot.org ([127.0.0.1]) by localhost (mail.u-boot.org [127.0.0.1]) (amavis, port 10024) with ESMTP id Y0UwRLZ2DL-Q for ; Tue, 6 Jan 2026 15:04:18 -0700 (MST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767737058; bh=WF+B8BUb1UAD/9XvlxrKxoTmwhTWNOqgfhmTNRgliKQ=; h=From:To:Date:CC:Subject:List-Id:List-Archive:List-Help:List-Owner: List-Post:List-Subscribe:List-Unsubscribe:From; b=lSbwAWUVHOAsukh7AClkQgU5TbATWVCX9N+mQpHmFGOicj1v9OiL1UKFHKo4dtEoR YbzhUv1hThEZOoUkgqzuXCGQjO27Wf0R+DvOSU9vaThVEgc2z9dcZ0OoePTVKSTgQD EZP19TOQN9owCGVBdDW5G1wfaATVOONpRXwrU8o8l3IxEZPvIl324je2j9PlW5TY93 /O8depiyi+A5ZbRc0PUhZZlZCdxBbcy3i74kXP+1K4eQVfsAV71lGd1EuFtnzXDFYI WzMLZLMBMdRKL53LLC5Vq5j3Lt8bCdgmcKYddU+yp25s6ghfmmknt46dMk4uPBU/hk 7st0MCsXxUHKg== Received: from mail.u-boot.org (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id CBA5869146 for ; Tue, 6 Jan 2026 15:04:18 -0700 (MST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767737057; bh=mWX+24wLfjK8XBsRHfB7ppCwTcP6IjSwWShRNbBFujg=; h=From:To:Cc:Subject:Date:From; b=UnvDAbKlWArquvtCII59cJI+5MwCyyfc9f+HhHr7vceqXzsnQd84Gl+r0z6uuiYUx ymfKflMn15y96v7BDQJe9Lpqrtc2SQHCfRv+wfhfsN2JItWg/gjFgkZuOboTIs/A5/ e/m2lp4GRic/nEvaVXUgcaWwd+yVU/M1opInj+IxdPITvY1HqDqiJh+9TkHxpwPQrc /81sB/HJzZmFnkWb+s4R3hwfQr2KyGk0jBN1V/lDEwTCvHvYA37bD/xOOM21wrUabf 2j4vAPafdynqq/Zjp9CzHgkmHvztUx7Aj5bDoqB8s3rys6VMswo+dtbXgV+NW0gW32 tPeg9mur0nZHQ== Received: from localhost (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id 4A32169145; Tue, 6 Jan 2026 15:04:17 -0700 (MST) X-Virus-Scanned: Debian amavis at Received: from mail.u-boot.org ([127.0.0.1]) by localhost (mail.u-boot.org [127.0.0.1]) (amavis, port 10026) with ESMTP id PnFbySP_HFT8; Tue, 6 Jan 2026 15:04:17 -0700 (MST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767737053; bh=e2biwYPKuOyWUAmdjo0wr0hIIypVk9zwLDyr6+6LLcw=; h=From:To:Cc:Subject:Date:From; b=nr5HT9+s99YhK1i9P0eqoWIxE7HOzNgyZhWfHJ5JYDzwRKejFywnsi+lKBTTapsKr KraWT7KXIrzRjUx6kxV6ERW/bhf5sJKDkHbh1wtuKNUSSrIV6WZtUzPoK67pfw733V +wpnFRFevd9CyBIQKMDZJ86P49yaSfSuwkLFZ96n/srM88Ssl6hYLl/8cvMJDrERPh 0yp4t/+swqXpDrJuHVpHvzaYaQx5JSPXVhnKijHT0iRDpApUxcFuMD7zdcxItKV59l Y1LjMLlSSH6YDdZv3Hx/mV+oYSe2+1I5+YoA1DgLuyGut/f//bUfcF9omoaAaRhilg qmQdX5+H54W9A== Received: from u-boot.org (unknown [73.34.74.121]) by mail.u-boot.org (Postfix) with ESMTPSA id D1580690CB; Tue, 6 Jan 2026 15:04:12 -0700 (MST) From: Simon Glass To: U-Boot Concept Date: Tue, 6 Jan 2026 15:03:18 -0700 Message-ID: <20260106220403.2915630-1-sjg@u-boot.org> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 Message-ID-Hash: W62DHUUC6FMP4DHZEOIGWQM4T2WPYZ55 X-Message-ID-Hash: W62DHUUC6FMP4DHZEOIGWQM4T2WPYZ55 X-MailFrom: sjg@u-boot.org X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: Simon Glass X-Mailman-Version: 3.3.10 Precedence: list Subject: [Concept] [PATCH 00/29] buildman: Clean up builder.py List-Id: Discussion and patches related to U-Boot Concept Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: From: Simon Glass This series cleans up builder.py to address pylint warnings and improve code structure. The first part fixes various pylint issues: - Remove unused imports and variables - Fix indentation and line-length issues - Replace deprecated setDaemon() with daemon property - Add explicit encoding to open() calls - Rename variables that shadow builtins - Convert to f-strings - Add module docstring and parameter types - Initialise all attributes in __init__() - Mark unused function arguments The second part refactors large functions to reduce complexity: - Extract helper methods from get_build_outcome() - Extract helper methods from get_result_summary() - Split print_size_summary() into smaller functions - Split print_result_summary() into smaller functions - Move regex patterns to module-level constants - Extract thread setup from __init__() Simon Glass (29): buildman: Remove unused imports from builder.py buildman: Fix bad indentation in builder.py buildman: Replace setDaemon() with daemon property buildman: Add explicit encoding to open() calls buildman: Remove unused variables from builder.py buildman: Rename variables that shadow builtins buildman: Fix minor pylint warnings in builder.py buildman: Convert to f-strings in builder.py buildman: Add module docstring and parameter types buildman: Initialise all attributes in Builder.__init__() buildman: Fix miscellaneous pylint warnings in builder.py buildman: Mark unused function arguments in builder.py buildman: Fix line length and return-value pylint warnings buildman: Refactor get_build_outcome() in builder.py buildman: Refactor get_result_summary() in builder.py buildman: Refactor print_size_summary() in builder.py buildman: Refactor _calc_size_changes() in builder.py buildman: Refactor print_size_summary() more in builder.py buildman: Refactor print_result_summary() in builder.py buildman: Split out environment and config display buildman: Split out target loop in _show_config_changes() buildman: Split out arch config summary display buildman: Return named tuple from _classify_boards() buildman: Extract arch results display buildman: Extract not-built display buildman: Extract IDE output in print_result_summary() buildman: Disable R0902/R0903 pylint warnings in builder.py buildman: Move regex patterns to module-level constants buildman: Extract thread setup from __init__() in builder.py tools/buildman/builder.py | 1513 ++++++++++++++++++++++--------------- 1 file changed, 909 insertions(+), 604 deletions(-)