Update tools/check_board_header.py

so that it handles pico_cmake_set_default the same way as cmake/generic_board.cmake does
This commit is contained in:
Andrew Scheller 2024-08-19 14:49:39 +01:00
parent 568377d294
commit 34e3cb956c

View File

@ -99,15 +99,7 @@ def read_defines_from(header_file, defines_dict):
# check all uppercase
if name != name.upper():
raise Exception("{}:{} Expected \"{}\" to be all uppercase".format(board_header, lineno, name))
# check for multiply-defined values
if name in cmake_default_settings:
if cmake_default_settings[name].value != value:
if board_header_basename != "datanoisetv_rp2040_dsp.h":
raise Exception("{}:{} Conflicting values for pico_cmake_set_default {} ({} and {})".format(board_header, lineno, name, cmake_default_settings[name].value, value))
else:
if show_warnings:
warnings.warn("{}:{} Multiple values for pico_cmake_set_default {} ({} and {})".format(board_header, lineno, name, cmake_default_settings[name].value, value))
else:
if name not in cmake_default_settings:
cmake_default_settings[name] = DefineType(name, value, None, lineno)
continue