Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36626 )
Change subject: util/lint/kconfig_lint: Handle glob prefix and suffix ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
Looks good, apart from the escaping, see inline comments...
https://review.coreboot.org/c/coreboot/+/36626/1/util/lint/kconfig_lint File util/lint/kconfig_lint:
https://review.coreboot.org/c/coreboot/+/36626/1/util/lint/kconfig_lint@1231 PS1, Line 1231: $dir_glob_prefix My google-foo says we should escape the variables (in case they contain regexp-special characters), e.g. \Q$dir_glob_prefix\E
https://review.coreboot.org/c/coreboot/+/36626/1/util/lint/kconfig_lint@1231 PS1, Line 1231: looks like this file uses spaces only for indentation