Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tools/ci: Switch kconfig-frontend to kconfiglib #8504

Merged
merged 5 commits into from
Feb 17, 2023

Conversation

xiaoxiang781216
Copy link
Contributor

Summary

Follow #8494
kconfiglib can catch more error than kconfig-frontend: #8485

Impact

build system

Testing

CI

Purge warning:
Kconfig:249: warning: the 'modules' option is not supported.
Let me know if this is a problem for you, as it wouldn't be that
hard to implement. Note that modules are supported -- Kconfiglib
just assumes the symbol name MODULES, like older versions of the
C implementation did when 'option modules' wasn't used.

Signed-off-by: chao an <[email protected]>
@GUIDINGLI GUIDINGLI merged commit fff0e58 into apache:master Feb 17, 2023
@xiaoxiang781216 xiaoxiang781216 deleted the kconfiglib branch February 17, 2023 08:18
@davids5
Copy link
Contributor

davids5 commented Feb 17, 2023

@xiaoxiang781216 @anchao - this broke "make menuconfig" see #8557 (comment)

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.

Kconfig:249: warning: the 'modules' option is not supported
5 participants