Fork of https://github.com/zephyrproject-rtos/Kconfiglib.git https://git.mcdonnell.dev/Kconfiglib.git/atom?h=v9.4.1 2018-08-17T19:42:29Z 2018-08-17T19:42:29Z Ulf Magnusson ulfalizer@gmail.com 2018-08-17T19:37:26Z urn:sha1:5d973e81c192b8e214437317eb3834fba93f1f0f - Commit 8a3999b ("Fix $srctree logic for the top-level Kconfig file") fixes a major braino where the top-level Kconfig file passed to Kconfig.__init__() wasn't looked up relative to $srctree, breaking out-of-tree operation for e.g. menuconfig. All Kconfig files are now consistently looked up relative to $srctree. - Misc. internal code cleanup 2018-08-17T19:35:42Z Ulf Magnusson ulfalizer@gmail.com 2018-08-17T19:04:43Z urn:sha1:8a3999bc708e8468ff79665e3cbdfccd603160e1 Due to a major design braino, the top-level Kconfig file passed to Kconfig.__init__() wasn't looked up relative to $srctree, breaking out-of-tree usage for e.g. menuconfig. With this change, Kconfig files are consistently looked up relative to $srctree, which makes a lot more sense. Also remove note re. loading a subset of Kconfig files. Saying that the top-level file and all source'd Kconfig files are looked up relative to $srctree should make the behavior clear enough. 2018-08-14T08:29:24Z Ulf Magnusson ulfalizer@gmail.com 2018-08-14T08:00:53Z urn:sha1:b7be018fba77db3a9ff11faf980c600d4bbb2fa8 += also does an in-place modification for lists, and it's a bit faster. Also get rid of an 'if node.defaults' tests. Both symbols and choices can have defaults, and it's not worthwhile as an optimization either. 2018-08-12T06:22:46Z Ulf Magnusson ulfalizer@gmail.com 2018-08-12T06:22:46Z urn:sha1:43480de6b2e50dc424232e9b9c97b19502f3e354 node.item is already available as 'sym'. 2018-08-12T05:25:17Z Ulf Magnusson ulfalizer@gmail.com 2018-08-12T05:25:17Z urn:sha1:40d45c0284838ce4df3ae8d36a867fa0d754e3a3 select_val was only used in a single place, and there's no real harm in calculating expr_value(self.direct_dep) twice for a warning. 2018-08-11T05:05:28Z Ulf Magnusson ulfalizer@gmail.com 2018-08-11T05:05:28Z urn:sha1:eb3b4a1672a1c8a64f1a17b97e80f8ffdae3f3f6 Kconfiglib changes: - Commit 1f864b1 ("Support custom printing of symbols/choices in expressions") adds support for printing symbols and choices with a custom format in expressions. This makes it simpler to implement things like like turning all symbols/choices into links when generating documentation, and printing symbol values after symbols. - Commit 6aea8d0 ("Don't pass encoding= to Popen()") fixes an accidental Python 3.6+ requirement when using the $(shell) preprocessor function with Python 3. menuconfig changes: - Commit 8efc358 ("menuconfig: Omit tristate value hint for int/string/hex defaults") removes the boolean value hints to the right of defaults in the symbol information for string/int/hex symbols. Stuff like '- 74 (value: "n")' isn't helpful, and looks confusing. - Commit 3511f24 ("menuconfig: Make symbol information less spammy") makes the symbol information more compact in general, e.g. by skipping value hints where they aren't helpful ('FOO(=y)' instead of 'FOO(=y) (=y)'), and by shortening stuff like '(value: "y")' to just '(=y)'. 2018-08-11T02:20:53Z Ulf Magnusson ulfalizer@gmail.com 2018-08-11T00:54:27Z urn:sha1:6aea8d06b637ea55a2fa7d2949863adae5310ce2 Popen()'s 'encoding' parameter is Python 3.6+ only. Unfortunately, Popen()'s universal_newlines=True without 'encoding' will use the encoding from the environment. Do a manual version instead, so that we can still use the user-specified encoding (usually UTF-8). That might prevent problems on systems that are (poorly) configured to use the C locale. 2018-08-10T04:39:22Z Ulf Magnusson ulfalizer@gmail.com 2018-08-10T04:23:53Z urn:sha1:d0732cff9cef507d79d5c53fbcc648dd6f560ce5 - *_fn() prefixes on functions are a bit silly. It makes more sense for the expr_str() parameter, so keep it there. - Use *_expr_str() instead of just *_str(), to make it clearer that these deal with expressions. 2018-08-10T02:57:13Z Ulf Magnusson ulfalizer@gmail.com 2018-08-10T01:21:38Z urn:sha1:3511f244641b5c786ffb38c881e7b70982510d22 - Use '(=value)' instead of '(="value")' - Use '(=value)' instead of '(value: {})' for various headings, too - Don't bother showing the value hint for the entire expressions when it's just a single symbol. The value of the symbol itself is already shown. - Only put quotes around the value for string symbols at the top of the symbol info display - Deindent expressions split over multiple lines a bit 2018-08-10T02:57:13Z Ulf Magnusson ulfalizer@gmail.com 2018-08-10T01:15:33Z urn:sha1:8efc358b6468834c7913cdf213e4aba05aeb14f7 Previously, defaults for string/int/hex symbols showed up as e.g. '17 (value: "n")' in the symbol information, which looks confusing. Only display the hint for bool/tristate symbols to fix it. For "non-literal" symbols, _expr_str() still takes care of showing the value (e.g. 'FOO(="bar")').