Skip to content

Commit dd2024c

Browse files
committed
scripts/check-logs.py: Treat empty string values as 'n' for config check
Commit 3e2f19e ("generator: yml: Reset CONFIG_EFI_SBAT_FILE for Fedora configurations") used '=n' to reset a string value to its default due to a limitation in TuxSuite / TuxMake. This causes a mismatch between the value provided and the value in the config, resulting in an error from the config check: ERROR: CONFIG_EFI_SBAT_FILE=n not found in .config! If the value is empty, treat it as 'n' for the sake of this check. Signed-off-by: Nathan Chancellor <[email protected]>
1 parent bb0a270 commit dd2024c

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

scripts/check-logs.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,11 @@ def check_built_config(build):
149149
state = None
150150
if '=' in line:
151151
name, state = line.split('=', 1)
152+
# Treat empty string values as 'n', as TuxSuite only supports y|m|n
153+
# for overriding configuration options, so 'n' may be used to reset
154+
# a value
155+
if state == '""':
156+
state = 'n'
152157
elif line.startswith("# CONFIG_"):
153158
name, state = line.split(" ", 2)[1:]
154159
if state != "is not set":

0 commit comments

Comments
 (0)