From c4fc25463e326ab2de553088f53fe8221475a4bf Mon Sep 17 00:00:00 2001
From: =?utf8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>
Date: Thu, 1 Jun 2023 10:39:54 +0200
Subject: [PATCH] ukify: make code pylint clean

The linter is imperfect, but it is useful as a very quick
check for typos and other silly mistakes. Add a few annotations
and do one small change to make it think the code is perfect.
---
 src/ukify/ukify.py | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/ukify/ukify.py b/src/ukify/ukify.py
index 19896afac37..88189d272d9 100755
--- a/src/ukify/ukify.py
+++ b/src/ukify/ukify.py
@@ -19,7 +19,9 @@
 # pylint: disable=missing-docstring,invalid-name,import-outside-toplevel
 # pylint: disable=consider-using-with,unspecified-encoding,line-too-long
 # pylint: disable=too-many-locals,too-many-statements,too-many-return-statements
-# pylint: disable=too-many-branches,fixme
+# pylint: disable=too-many-branches,too-many-lines,too-many-instance-attributes
+# pylint: disable=too-many-arguments,unnecessary-lambda-assignment,fixme
+# pylint: disable=unused-argument
 
 import argparse
 import configparser
@@ -438,7 +440,7 @@ def call_systemd_measure(uki, linux, opts):
 def join_initrds(initrds):
     if len(initrds) == 0:
         return None
-    elif len(initrds) == 1:
+    if len(initrds) == 1:
         return initrds[0]
 
     seq = []
@@ -478,6 +480,9 @@ def pe_add_sections(uki: UKI, output: str):
             pe.FILE_HEADER.IMAGE_FILE_LOCAL_SYMS_STRIPPED = True
 
     # Old stubs might have been stripped, leading to unaligned raw data values, so let's fix them up here.
+    # pylint thinks that Structure doesn't have various members that it has…
+    # pylint: disable=no-member
+
     for i, section in enumerate(pe.sections):
         oldp = section.PointerToRawData
         oldsz = section.SizeOfRawData
@@ -745,6 +750,7 @@ class ConfigItem:
     ) -> None:
         "Set namespace.<dest>[idx] to value, with idx derived from group"
 
+        # pylint: disable=protected-access
         if group not in namespace._groups:
             namespace._groups += [group]
         idx = namespace._groups.index(group)
@@ -1068,7 +1074,7 @@ def apply_config(namespace, filename=None):
     # Fill in ._groups based on --pcr-public-key=, --pcr-private-key=, and --phases=.
     assert '_groups' not in namespace
     n_pcr_priv = len(namespace.pcr_private_keys or ())
-    namespace._groups = list(range(n_pcr_priv))
+    namespace._groups = list(range(n_pcr_priv))  # pylint: disable=protected-access
 
     cp = configparser.ConfigParser(
         comment_prefixes='#',
-- 
2.47.3