Attention is currently required from: Felix Singer, Nico Huber, Martin Roth, Angel Pons, Julius Werner. Michael Niewöhner 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: Code-Review+1
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/56410/comment/e47d1c13_3f2412fb 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. Martin, I don't see any direct offense in your direction, here. It's just facts and a try to make things better.
If anyone would like to take a stab at writing a better linter, I'd welcome the effort. Again, if the coreboot community doesn't think that linting the Kconfigs is a good idea,
Nobody said that. My parent change - that is merged already - covers parts of the stuff that is dropped here.
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.)
@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.