Fork of https://github.com/zephyrproject-rtos/Kconfiglib.git https://git.mcdonnell.dev/Kconfiglib.git/atom?h=v12.8.0 2019-06-24T06:57:38Z 2019-06-24T06:57:38Z Ulf Magnusson ulfalizer@gmail.com 2019-06-24T06:18:06Z urn:sha1:0ca8e78ebaed9ded586162753e8224f8ce7ef4a2 Getting defaults, select, implies, and ranges without dependencies propagated to conditions can be useful for symbols and choices as well, e.g. when generating standalone documentation for properties. 2019-06-23T00:52:26Z Ulf Magnusson ulfalizer@gmail.com 2019-06-23T00:29:14Z urn:sha1:7b97da6821a45ce34e731df68980dea0983cf18c Do 'add = fields.append'. Pointless as an optimization, but shaves some lines, and obvious in context. Also add test coverage for __repr__() for string symbols with user values. 2019-06-03T04:50:06Z Ulf Magnusson ulfalizer@gmail.com 2019-06-02T23:55:50Z urn:sha1:bf36f5de0f49d56975b1a844e25484ec385dc971 A wart of the warning control API (enable/disable_*_warnings()) is that the current warning settings can't be queried. Querying warning settings is useful in functions that want to temporarily enable/disable some warning and then put things back to how they were. kconfiglib.load_allconfig() ran into this, for example. Make the internal warning control variables public (improve the naming at the same time), and encourage just setting them directly. Keep the old API for backwards compatibility. Also remove _warn_redun_assign() and _warn_override(). They're trivial and were called in a single place. 2019-06-01T04:21:54Z Ulf Magnusson ulfalizer@gmail.com 2019-06-01T02:15:04Z urn:sha1:92791a3fe15bb2d5c14039fbd531be73255f0c6d Sourcing a file with an absolute path and using rsource in it triggered a relpath() between the absolute path and $srctree. Since e.g. symlink/../bar/ = bar/ is not guaranteed for symlinks, this could lead to the rsource'd file not being found if $srctree pointed to a symlink. Switch to a simpler, more textual method for stripping $srctree from glob results, which should be robust against symlink shenanigans. This also makes the code a bit easier to follow. Discovered by Marc Herbert. Piggyback some minor cleanup. 2019-05-29T00:47:44Z Ulf Magnusson ulfalizer@gmail.com 2019-05-29T00:45:35Z urn:sha1:15de4c0c9924cfeef1553fde21cb9b9e288f6238 Like was done a long time ago in kconfiglib.py. These functions are not part of a standalone API. Keep the docstrings for the compatibility tests, because those get printed as the tests run. 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-24T10:21:54Z Ulf Magnusson ulfalizer@gmail.com 2019-05-24T10:20:18Z urn:sha1:e806f26a075ba52eca01567c7be7f0b4eba91df5 Ye olde code.