search

ulfalizer / Kconfiglib

A flexible Python 2/3 Kconfig implementation and library
ISC License
448 stars 160 forks source link

Add rudimentary support for modules property #119

Open helmutg opened 2 years ago

helmutg commented 2 years ago

In linux commit 6dd85ff178cd76851e2184b13e545f5a88d1be30, Linux Torvalds changed "option modules" to plain "modules" since it was the only option left. kconfiglib does not have much support for either besides parsing it and suppressing warnings when it is applied to the 'MODULES' symbol. Mirror this behaviour for the newer "modules" property.

Fixes: #106

envzhu commented 1 year ago

This patch is very helpful for me. Thank you, @helmutg .

jackrosenthal commented 3 weeks ago

Rather unfortunately, it appears the original author of Kconfiglib disappeared. The Zephyr project has been granted ownership of the PyPI package via PEP-541 request, and I'm going to do a release based on a new repository in the zephyrproject-rtos organization here: https://github.com/zephyrproject-rtos/Kconfiglib

:arrow_right: Action required: @helmutg Are you able to re-open your pull request on the new repository?

helmutg commented 3 weeks ago

@jackrosenthal, I tried but github seems to be unable to draw a connection between the repositories. If I attempt to open a MR on my own, I cannot select the zephyrproject and if I open a MR on the zephyrproject I cannot select my fork. I suspect someone has to push my branch to the zephyrproject instance and then open a MR from that.

I do not intend to further work on this nor improve my MR. I will not object to closing it.