Fork of https://github.com/zephyrproject-rtos/Kconfiglib.git https://git.mcdonnell.dev/Kconfiglib.git/atom?h=v1.0.5 2017-10-01T04:52:31Z 2017-10-01T04:52:31Z Ulf Magnusson ulfalizer@gmail.com 2017-10-01T04:28:02Z urn:sha1:2a2fcbd404878d539683178d98c67c983b5f61c9 Just like for other properties, conditions on ranges get local 'depends on' and parent dependencies propagated to them. Oversight. Did not trigger any deviations for the kernel defconfigs. Pretty specific circumstances were required for breakage, like a symbol depending on the particular value of a symbol with a 'range' and parent deps 'n', or a symbol with ranges being defined in multiple locations with different parent deps. (There is one symbol that both has ranges and is defined in multiple locations: BCH_CONST_M. The second definition adds a default rather than a range though.) 2017-10-01T00:50:29Z Ulf Magnusson ulfalizer@gmail.com 2017-10-01T00:09:34Z urn:sha1:ab58c2388128df0dfe361b7d1a795c520cbfa55d Slightly modifying _STRING_LEX allows the (common) symbol reference case to be detected earlier and removes some assignments. Also modify some comments and air things out a bit. 2017-09-30T20:12:48Z Ulf Magnusson ulfalizer@gmail.com 2017-09-30T18:46:02Z urn:sha1:2c062711d974be25346776fcb30ab1310cd896e2 Calculate as needed instead, using _get_expr_syms(). Simplifies the parsing code. Also rename _get_expr_syms*() to _expr_syms*() and make it append to a list. Handy for the new way it's being used. 2017-09-30T16:43:42Z Ulf Magnusson ulfalizer@gmail.com 2017-09-30T16:43:42Z urn:sha1:890e5b6bec0523a7de57ebb9c8fd4caa7c2c5905 Calculate as needed instead. Old cruft. 2017-09-30T16:09:37Z Ulf Magnusson ulfalizer@gmail.com 2017-09-30T16:08:39Z urn:sha1:11dc7657e2ddc615f38cc87c4c8ae6e1be939eba Calculate it as needed instead. It's not used internally. Also move get_referenced_symbols() to the Item base class, as the logic is identical for symbols/choices/menus/comments. Finally a non-silly use for it. 2017-09-30T15:43:10Z Ulf Magnusson ulfalizer@gmail.com 2017-09-30T04:13:00Z urn:sha1:9a622874419fac263c32f55786760a266a7f7f32 - Menu/Comment._dep_expr and Symbol/Choice._menu_dep are really the same thing internally in the C implementation (the dependency for the "menu node", which is also used to implicitly create menus), and it simplifies the code to treat them like that here too. - Reuse the calculated _menu_dep (which has parent and 'depends on' dependencies) when propagating dependencies to properties. - Get rid of the temporary lists. It's now easy to use plain 'for' loops to add the properties. - Catch attempts to use select/imply from menus and choices early and error out. - Add comments clarifying which variables are only used when displaying items. 2017-09-29T20:26:02Z Ulf Magnusson ulfalizer@gmail.com 2017-09-29T20:26:02Z urn:sha1:d1b32ad7890b5f32aba4db5ef798877fd333d573 There's already a "misc." section. Also fix typo in docstring. 2017-09-29T18:07:39Z Ulf Magnusson ulfalizer@gmail.com 2017-09-29T15:30:34Z urn:sha1:250b370cb5a1d40d7c0e2a4df7275964c1d24fea Minor fixes: - Even with print_undef_assign=True, "# CONFIG_NOT_DEFINED is not set" would not generate a warning. - Change the warning when a variable is set more than once in the .config to be a bit more explicit. - Only de-quote and unescape assignments to string variables, which is nicer and matches the C implementation. 2017-09-29T14:45:47Z Ulf Magnusson ulfalizer@gmail.com 2017-09-29T14:26:05Z urn:sha1:c54db8e7c9cde50a2b13cebe256cfd907b45d762 Rearrange a bit and document that a trailing newline does not need to be added. Clean up the tests a bit too. 2017-09-29T12:48:54Z Ulf Magnusson ulfalizer@gmail.com 2017-09-29T11:56:20Z urn:sha1:7cbfa4711033e7f0d6b730a2ad22fd1caa84beea Previously we'd crash instead of printing the warning, because we didn't pass 'filename' and 'linenr' to _stderr_msg(). This code path currently can't be reached via load_config() and needs an explicit set_user_value(), so the bug went undetected. Fix by breaking out a separate _warn_undef_assign() that defaults 'filename' and 'linenr' to None, similar to _warn().