diff mbox series

[Concept,05/20] buildman: Fix import order in control.py

Message ID	20260316154733.1587261-6-sjg@u-boot.org
State	New
Headers	From: Simon Glass <sjg@u-boot.org> To: U-Boot Concept <concept@u-boot.org> Date: Mon, 16 Mar 2026 09:47:10 -0600 Message-ID: <20260316154733.1587261-6-sjg@u-boot.org> In-Reply-To: <20260316154733.1587261-1-sjg@u-boot.org> References: <20260316154733.1587261-1-sjg@u-boot.org> MIME-Version: 1.0 Message-ID-Hash: CRTFAEUMWISTX3FCWGEX3TCCEA2DIT5P CC: Simon Glass <sjg@chromium.org> Precedence: list Subject: [Concept] [PATCH 05/20] buildman: Fix import order in control.py Archived-At: <https://lists.u-boot.org/archives/list/concept@u-boot.org/message/CRTFAEUMWISTX3FCWGEX3TCCEA2DIT5P/> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit
Series	buildman: Add distributed builds \| [Concept,00/20] buildman: Add distributed builds [Concept,01/20] u_boot_pylib: Add stdin_data support to run_pipe() [Concept,02/20] u_boot_pylib: Support passing modules to run_test_suites() [Concept,03/20] buildman: Remove unused imports from builder.py [Concept,04/20] buildman: Fix pylint warnings in test_builder.py [Concept,05/20] buildman: Fix import order in control.py [Concept,06/20] buildman: Initialise _timestamps to an empty deque [Concept,07/20] buildman: Add thread_class param to Builder [Concept,08/20] buildman: Add handle_signals param to Builder [Concept,09/20] buildman: Add lazy thread setup to Builder [Concept,10/20] buildman: Split build_boards() into two methods [Concept,11/20] buildman: Add distributed-build attributes to Builder [Concept,12/20] buildman: Cache the kconfig-changed check per commit [Concept,13/20] buildman: Add dynamic job-count setting [Concept,14/20] buildman: Pass NPROC to make for LTO parallelism [Concept,15/20] buildman: Add remote machine probing for distributed builds [Concept,16/20] buildman: Install toolchains on remote machines [Concept,17/20] buildman: Add worker mode for distributed builds [Concept,18/20] buildman: Add boss module for driving remote workers [Concept,19/20] buildman: Wire up distributed builds with WorkerPool

Commit Message

Simon Glass March 16, 2026, 3:47 p.m. UTC

  From: Simon Glass <sjg@chromium.org>

Move the patman import after u_boot_pylib imports to satisfy pylint's
C0411 (wrong-import-order) check.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 tools/buildman/control.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff mbox series

Patch

diff --git a/tools/buildman/control.py b/tools/buildman/control.py
index 7fc0d20574a..989057db60e 100644
--- a/tools/buildman/control.py
+++ b/tools/buildman/control.py
@@ -22,7 +22,6 @@  from buildman import toolchain
 from buildman.builder import Builder
 from buildman.outcome import DisplayOptions
 from buildman.resulthandler import ResultHandler
-from patman import patchstream
 import qconfig
 from u_boot_pylib import command
 from u_boot_pylib import gitutil
@@ -30,6 +29,8 @@  from u_boot_pylib import terminal
 from u_boot_pylib import tools
 from u_boot_pylib.terminal import print_clear, tprint
 
+from patman import patchstream
+
 TEST_BUILDER = None
 
 # Space-separated list of buildman process IDs currently running jobs
@@ -564,7 +565,8 @@  def run_builder(builder, commits, board_selected, display_options, args):
                                   args.threads, args.jobs))
 
     builder.set_display_options(
-        display_options, args.filter_dtb_warnings, args.filter_migration_warnings)
+        display_options, args.filter_dtb_warnings,
+        args.filter_migration_warnings)
     if args.summary:
         builder.commits = commits
         builder.result_handler.show_summary(