Fork of https://github.com/zephyrproject-rtos/Kconfiglib.git https://git.mcdonnell.dev/Kconfiglib.git/atom?h=v10.4.0 2018-08-29T06:09:31Z 2018-08-29T06:09:31Z Ulf Magnusson ulfalizer@gmail.com 2018-08-29T06:09:23Z urn:sha1:fbd58f9789d27207e30ab2c126773bc6894a15d6 Show-all mode does not need to be enabled when jumping to an invisible symbol with visible children from an implicit submenu, because the invisible symbol will be shown anyway in that case. Explicitly check whether the jumped-to node would be shown instead, and enable show-all mode otherwise. 2018-08-29T05:49:39Z Ulf Magnusson ulfalizer@gmail.com 2018-08-29T05:01:17Z urn:sha1:d1e2a652de33e1b928515277452fd939a290f5ed Currently, the symbol BAR below (which ends up indented in an implicit submenu) is shown only if DEP is non-n (or if show-all mode is enabled): config FOO bool "foo" if DEP default y config BAR bool "bar" if FOO This is bad, because it hides visible symbols from the interface. The assumption was that an implicit submenu (which is only created if the parent has a prompt) would never have visible items when the root item is invisible, but prompt-specific conditions and select/imply can break that assumption. Fix it by always showing implicit submenus with visible nodes, along with the parent node. If the parent node is invisible, show it in red, like in show-all mode (which happens automatically). That's probably better than having mysteriously indented nodes when the parent is invisible. 2018-08-29T01:18:56Z Ulf Magnusson ulfalizer@gmail.com 2018-08-29T01:18:56Z urn:sha1:739acb609da605eb90a2806d97e40c330f91bed7 2018-08-29T01:15:42Z Ulf Magnusson ulfalizer@gmail.com 2018-08-29T01:15:18Z urn:sha1:088e17a47d8efaee06b1f5e2fabaa939dc70e798 Just checking whether the default value expression is a single symbol correctly handles the string/int/hex cases too, because that case will always hit for those. D'oh. 2018-08-29T01:03:32Z Ulf Magnusson ulfalizer@gmail.com 2018-08-29T01:03:32Z urn:sha1:e435ecc6f367a2d0f7159bd3c7c0f1aca8dbb982 Used in two spots. 2018-08-29T00:50:46Z Ulf Magnusson ulfalizer@gmail.com 2018-08-29T00:50:46Z urn:sha1:64b734a930a7670671ef5fdc21c12f2b11b04409 The global should be called _stdscr, and then the names don't clash. Leftover. 2018-08-29T00:22:25Z Ulf Magnusson ulfalizer@gmail.com 2018-08-29T00:22:25Z urn:sha1:15d07b1dceb5f651c838356378b350251ccf90a2 Add info to module docstring. 2018-08-28T22:52:05Z Ulf Magnusson ulfalizer@gmail.com 2018-08-28T22:12:32Z urn:sha1:84bd8634bb5de0895ff3354c394f43b923acb230 Add an alternative, less yellow, more 'make menuconfig'-like color scheme, contributed by Mitja Horvat (pinkfluid). The new theme is selected by setting the MENUCONFIG_THEME environment variable to 'aquatic'. Make it so that custom themes can override just the styles they want to change from the default theme. Perhaps some non-hardcoded theming mechanism could be added later, but keep it simple for now. Piggyback some related changes: - Make the selected item bold in the default theme, like the aquatic theme does. - Allow the style for the edit box in the jump-to dialog to be configured separately from the style for edit boxes in pop-up dialogs. That makes it possible to avoid some color clashes. - Remove a redundant style setting for the jump-to edit box. The window itself is already created with the correct style. 2018-08-26T01:04:30Z Ulf Magnusson ulfalizer@gmail.com 2018-08-26T00:34:23Z urn:sha1:ddfd5df84b4b7871ea16d76e233c11c8b45738d9 Promptless choices can appear "legitimately" if you define a named choice in multiple locations to add on some symbols (which is broken in the C tools though). Prior to this fix, the promptless choice would get flattened, with the choice symbols appearing in the same menu as the (invisible) choice. This looks confusing. Skip flattening promptless choices to fix it. 2018-08-25T23:51:09Z Ulf Magnusson ulfalizer@gmail.com 2018-08-25T21:22:40Z urn:sha1:2320b7adb703c22e44ec9716a0d2605e80a02783 Some general cleanup: - Handle the iteration with the new Kconfig.node_iter() helper. This makes some function so short that they become pointless. Have just main() + oldconfig(node) + small helper functions. - Use _name_and_loc_str(sc) for choices too, so that all locations get reported for named choices defined in multiple locations - Rewrite the intro to be less wordy and remove the sample session (leftover from when oldconfig.py was in examples/, and not that exciting) - Print "Updated configuration written to..." instead of "Configuration saved to..." - Various other nits