search

ulfalizer / Kconfiglib

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

menuconfig: Avoid crashing when leaving a menu not shown in parent #94

Open leandrolanzieri opened 4 years ago

leandrolanzieri commented 4 years ago

This adds a catch to an exception that might occur when leaving a menu which is not shown by the parent menu. For an example of this refer to issue #93.

For the snippet given in #93 this ends up showing the top menu. I'm not sure if this is the expected behaviour (after all, the second definition of the choice is defined in the top menu), or 'Sub' menu should be shown. At least this seems to avoid the crash.

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: @leandrolanzieri Are you able to re-open your pull request on the new repository?

leandrolanzieri commented 1 week ago

@jackrosenthal see https://github.com/zephyrproject-rtos/Kconfiglib/pull/4