-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
feat(scripts): add default config generator #6522
Conversation
384c023
to
6d1d6f8
Compare
Please rebase on master to fix the CI issue. |
6d1d6f8
to
42f2c9d
Compare
@XuNeo could you take a look at this PR? |
Sorry for the delay. Last time I tested, it has some problem, I can try to reproduce it. In the meanwhile, I discussion with some fellows, and question why not use Kconfig to generate the config files? |
Kconfig has some important limitations. E.g. we can't handle these. |
@XuNeo please try to reproduce. Is Kconfig able to cover our use case? I'm not aware of a mode where we are able to generate a "full list" of the macro definitions with some default values being overwritten. |
Hi, Everyone! We are going to need to coordinate. @becseya , I see your new indents in Kind regards, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's working for me
Sorry for the late response. I just tried the tools now and it works for me too. |
What Kconfig can't do is configurations needs preprocessor. Other than that, it works great and has lots of tools. RTOS NuttX, Zephyr, RT-Thread etc and linux all support Kconfig. For preprocessor, the solution in NuttX is to define it in Makefile. |
Co-authored-by: Liam <30486941 [email protected]>
I'd like to go with a simple and platform independent solution first. I think it's the safest bet. We can improve it later if needed. |
This PR adds a python script that can be used to update the lv_conf.h file of an existing project based on "lv_conf.default" parameters.
Do not squash please.