Attention is currently required from: Felix Singer, Martin Roth, Angel Pons, Julius Werner, Michael Niewöhner. Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56410 )
Change subject: [RFC] kconfig_lint: Drop overly restrictive rules about choice configs ......................................................................
Patch Set 5:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/56410/comment/657ebfe2_7344953a PS4, Line 19: On top, the linter treats every occurence of a `config` entry as a : symbol declaration, even when it's just setting a default or adding : selects.
I actually supported the idea of using kconfiglib, which can do what this linter does and more, but that idea was shot down.
Well, push an RFC and let's discuss it. I'd be happy to take a look and would probably even support it. (Even though, I know some people hating python... I don't.)
See the recent mailing list discussion about a general Python dependency. It's very very very unlikely to happen. The strongest supporter in that discussion actually provided the killing argument against adding more dependencies (they had to debug something for weeks because they had a bad binary in their build environment, AIUI).
I was quite excited when people told me about kconfiglib and what it can do. And even more unhappy when I learned that it's written in Python :-/
@ALL Let's come back to a objective discussion on that topic. This is still blocking development of others and we should start to solve that problem. Thank you all.
I want to try to keep the last check as you suggested. Will push something soon, hopefully.