Fork of https://github.com/zephyrproject-rtos/Kconfiglib.git https://git.mcdonnell.dev/Kconfiglib.git/atom?h=v13.0.0 2019-09-26T10:11:23Z 2019-09-26T10:11:23Z Ulf Magnusson ulfalizer@gmail.com 2019-09-26T10:11:23Z urn:sha1:4185b0b3193345f793ca7a5268fb67ab65e57655 Macros should still be expanded within nested parentheses. Test it. 2019-09-26T09:52:22Z Ulf Magnusson ulfalizer@gmail.com 2019-09-25T18:25:06Z urn:sha1:720ce88a839cc9966d4b5dfbcdbe868c33cb5965 As an oversight, there was no check for nested parentheses in macro arguments, making the preprocessor think the call ended after 'void)' in def_bool $(success,echo 'void foo(void) { asm inline (""); }' | $(CC) -x c - -c -o /dev/null) This broke the latest linux-next kernels, starting with commit eb111869301e1 ("compiler-types.h: add asm_inline definition"). I remember seeing this when going through the C code, but somehow forgot to put it in. Fix it, and clean up _expand_macro() a bit at the same time. Fixes: #76 2019-09-04T19:14:12Z Ulf Magnusson ulfalizer@gmail.com 2019-09-04T18:44:24Z urn:sha1:26e1db651e18fd65fe7374b404a30e20f601969f Just requires making Kconfig.filename/linenr public. 'lineno' would be a more standard name, but be consistent with MenuNode.linenr. 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-09T12:00:50Z Ulf Magnusson ulfalizer@gmail.com 2019-06-09T11:35:33Z urn:sha1:8453c52291eda9b9d954ac480fd062010f836a3f Weirdly formatted _T_HELP tokens are supported for compatibility with old versions of the C tools. Also add a lone '#' after the tricky help text. Some reconsidered refactoring choked on that. 2019-06-03T23:51:58Z Ulf Magnusson ulfalizer@gmail.com 2019-06-03T23:51:58Z urn:sha1:9d37b21392fb5aa732a2692c5aaa3836777cdd28 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-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: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