]> git.ipfire.org Git - thirdparty/mkosi.git/commitdiff
Stop returning namespace in parse_config()
authorDaan De Meyer <daan.j.demeyer@gmail.com>
Fri, 7 Apr 2023 13:27:51 +0000 (15:27 +0200)
committerDaan De Meyer <daan.j.demeyer@gmail.com>
Fri, 7 Apr 2023 13:27:51 +0000 (15:27 +0200)
We don't use the return value at all so let's make it None.

mkosi/config.py

index 2cae297d44dd40d89856090fb5f8ff5896319ba0..4597c2406b61cc9078dd0916bbfe41b285e36465 100644 (file)
@@ -693,7 +693,7 @@ class MkosiConfigParser:
     def __init__(self) -> None:
         self.lookup = {s.name: s for s in self.SETTINGS}
 
-    def parse_config(self, path: Path, namespace: argparse.Namespace) -> argparse.Namespace:
+    def parse_config(self, path: Path, namespace: argparse.Namespace) -> None:
         extras = path.is_dir()
 
         if path.is_dir():
@@ -733,7 +733,7 @@ class MkosiConfigParser:
                     setattr(namespace, s.dest, default)
 
                 if not match(s.dest, v, namespace):
-                    return namespace
+                    return
 
         parser.remove_section("Match")
 
@@ -750,15 +750,13 @@ class MkosiConfigParser:
             if path.parent.joinpath("mkosi.conf.d").exists():
                 for p in sorted(path.parent.joinpath("mkosi.conf.d").iterdir()):
                     if p.is_dir() or p.suffix == ".conf":
-                        namespace = self.parse_config(p, namespace)
+                        self.parse_config(p, namespace)
 
             for s in self.SETTINGS:
                 for f in s.paths:
                     if path.parent.joinpath(f).exists():
                         setattr(namespace, s.dest, s.parse(s.dest, str(path.parent / f), namespace))
 
-        return namespace
-
     def create_argument_parser(self) -> argparse.ArgumentParser:
         action = config_make_action(self.SETTINGS)