Skip to content

Conversation

philipanda
Copy link
Contributor

Importing the default platform configs in keywords.robot will fix language servers spamming warnings of undefined variables all over the place. The platform config defined in the ${CONFIG} variable is imported dynamically after all of the static imports, and takes precedence over them.

The documentation is confusing on this:

It is experimentally proven, to not be the case in a little test:
resource-precedence_log.zip

@philipanda philipanda requested a review from macpijan July 30, 2025 08:29
@philipanda
Copy link
Contributor Author

philipanda commented Jul 30, 2025

Sadly, it seems, that only the variables directly defined in the ${CONFIG} file are taking precedence over the default.robot variables. Variables defined in any includes from the Settings section of ${CONFIG} will not override the default.robot values:

${CHECK_POWER_LED_SUPPORT} was not overridden in this case, as can be seen in the error message and Log Variables in Prepare Test Suite.
fwupd-support_log.zip

In that case it might not be possible to solve this that easily.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant