From: Patrick Talbert <ptalbert(a)redhat.com>
redhat: Use a context manager in merge.py for opening the config file for reading
Signed-off-by: Clark Williams <williams(a)redhat.com>
diff --git a/redhat/configs/merge.py b/redhat/configs/merge.py
index blahblah..blahblah 100644
--- a/redhat/configs/merge.py
+++ b/redhat/configs/merge.py
@@ -25,14 +25,15 @@ def usage(msg):
# read a config file and return a dictionary of the contents
def read_config_file(cfgfile):
configs = {}
- for l in [ n.strip() for n in open(cfgfile).readlines()]:
- if len(l) == 0: continue
- if l.startswith("# CONFIG_"):
- configs[l.split()[1]] = l
- continue
- if l.startswith("CONFIG_"):
- configs[l.split('=')[0]] = l
- continue
+ with open(cfgfile) as f:
+ for l in [n.strip() for n in f.readlines()]:
+ if not l: continue
+ if l.startswith("# CONFIG_"):
+ configs[l.split()[1]] = l
+ continue
+ if l.startswith("CONFIG_"):
+ configs[l.split('=')[0]] = l
+ continue
return configs
--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2117