From: Trevor Gamblin <tgamblin@baylibre.com>
Date: Thu, 18 Jun 2026 20:36:27 +0000 (-0400)
Subject: scripts/patchtest: clean up startTestRun()
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b8f799c26d039339adf66bbb9639b18932e18a63;p=thirdparty%2Fopenembedded%2Fopenembedded-core.git

scripts/patchtest: clean up startTestRun()

- Initialize self.repo directly instead of in two steps
- Remove extra whitespace when setting error and failure members

Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
Signed-off-by: Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>
Signed-off-by: Ross Burton <ross.burton@arm.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---

diff --git a/scripts/patchtest b/scripts/patchtest
index 07c3ada6f7..ff8e8b11bd 100755
--- a/scripts/patchtest
+++ b/scripts/patchtest
@@ -61,20 +61,17 @@ def getResult(patch, mergepatch, logfile=None):
         skip        = 'SKIP'
 
         def startTestRun(self):
-            # let's create the repo already, it can be used later on
-            repoargs = {
-                "repodir": PatchtestParser.repodir,
-                "commit": PatchtestParser.basecommit,
-                "branch": PatchtestParser.basebranch,
-                "patch": patch,
-            }
-
-            self.repo_error    = False
-            self.test_error    = False
-            self.test_failure  = False
+            self.repo_error   = False
+            self.test_error   = False
+            self.test_failure = False
 
             try:
-                self.repo = PatchtestParser.repo = PatchTestRepo(**repoargs)
+                self.repo = PatchtestParser.repo = PatchTestRepo(
+                    patch=patch,
+                    repodir=PatchtestParser.repodir,
+                    commit=PatchtestParser.basecommit,
+                    branch=PatchtestParser.basebranch,
+                )
             except:
                 logger.error(traceback.print_exc())
                 self.repo_error = True