Fork of https://github.com/zephyrproject-rtos/Kconfiglib.git https://git.mcdonnell.dev/Kconfiglib.git/atom?h=v11.6.0 2019-05-25T16:04:15Z 2019-05-25T16:04:15Z Ulf Magnusson ulfalizer@gmail.com 2019-05-25T15:04:09Z urn:sha1:9f57bf933508901f07d22c75fca42e62145f5512 Pro: Looks cleaner and matches how definitions are usually written. Con: Makes it harder to discover that 'bool "foo"' is just shorthand for 'bool' + 'prompt "foo"' (though it's documented). The pros probably win. 2019-05-25T14:50:07Z Ulf Magnusson ulfalizer@gmail.com 2019-05-25T14:49:35Z urn:sha1:e1da7aa4d041d25e4fd5fd1f603f4c28d32b1392 These are constant symbols, which is why the quotes showed up. Special-case them to remove the quotes, to match what people expect. Internally, n/m/y without quotes is taken as a shorthand for "n"/"m"/"y". 2019-05-25T14:18:29Z Ulf Magnusson ulfalizer@gmail.com 2019-05-25T11:52:09Z urn:sha1:bb33eed5cc6bdce2b65aa570ecd9a400bcd9c180 Commit e81a77b ("Consistently put direct deps. last when propagating") makes the position of the direct deps. in property conditions predictable after dependency propagation, making it easy to strip them as needed. Use this to implement MenuNode.orig_{prompt,defaults,selects,implies,ranges}, which work like the non-orig_* versions but omit the direct deps. Use those in turn to omit the direct deps in Symbol/Choice.__str__(). The direct deps. (with propagated parent deps.) can still be seen after 'depends on ...', so there is no loss of information. This unclutters Kconfig definitions shown in menuconfig/guiconfig and in any generated documentation. The old output also had duplicated dependencies, though it doesn't matter for evaluation. Before: config A bool prompt "A" if DEP default y if FOO && DEP depends on DEP After: config A bool prompt "A" default y if FOO depends on DEP 2019-05-24T11:08:12Z Ulf Magnusson ulfalizer@gmail.com 2019-05-24T10:14:31Z urn:sha1:e81a77bdb0687b62f91d7dc4d5b84b443808c0a7 Preparation for some later additions. Previously, the 'visible if' deps appeared to the right of the 'depends on' deps. Now, the direct deps always appear last. With this change, the prompt tuple is only updated once for any given menu node too, which should be a tiny bit faster. 2019-05-23T07:49:42Z Ulf Magnusson ulfalizer@gmail.com 2019-05-23T07:49:42Z urn:sha1:95515d4cb09c02881bfbec1809c82f885ebd55c8 KconfigSyntaxError is just an older (bad) name. pydoc generates confusing documentation due to the "KconfigSyntaxError = KconfigError" assignment, so mention it in the docstring. 2019-05-22T06:52:08Z Ulf Magnusson ulfalizer@gmail.com 2019-05-22T06:50:08Z urn:sha1:f8978b04779aeae790b205a22f68cee0af2e4e5b Clearer. 2019-05-19T00:26:10Z Ulf Magnusson ulfalizer@gmail.com 2019-05-17T19:38:56Z urn:sha1:a75e073072399aa59fa52a0a0e02cbc1e9722dca 2019-05-11T12:27:25Z Ulf Magnusson ulfalizer@gmail.com 2019-05-11T12:10:43Z urn:sha1:95b80e16c4541089a8c5e33f73bbbac7a7f8ba8b - Mention that having no direct dependencies corresponds to having y direct dependencies - Remove the note about how direct_dep is used internally. It's also used for the select-with-unsatisfied-deps warning. - Remove the note about how checking direct_dep might be "redundant" since it's propagated. direct_dep can be useful in scripts still, and maybe it could scare people away from it. - Say 'surrounding menus and ifs' instead of of 'parent' in the 'dep' docstring - Fix two s/config/kconfig/ typos 2019-05-10T17:11:17Z Ulf Magnusson ulfalizer@gmail.com 2019-05-10T17:08:16Z urn:sha1:814e2decd190d78b588dd775c1531dacd6bc38c6 - Point out why dependencies from surrounding if's and menus are included - Clarify that .referenced isn't transitive. Only "direct" references are included. - Give a hint about direct dependencies and expr_items() Prompted by https://github.com/ulfalizer/Kconfiglib/issues/69. 2019-05-10T12:14:16Z Ulf Magnusson ulfalizer@gmail.com 2019-05-10T12:13:08Z urn:sha1:0f2b78c71e9fc2c9c1890e91dc37c1a853602847 It's undocumented and unguessable that Kconfig.top_node.filename/linenr gets set to the location of the 'mainmenu' statement (if any). Always set it to the first line of the top-level Kconfig instead, which should be more expected.