Fork of https://github.com/zephyrproject-rtos/Kconfiglib.git https://git.mcdonnell.dev/Kconfiglib.git/atom?h=v1.0.1 2017-09-24T12:03:41Z 2017-09-24T12:03:41Z Ulf Magnusson ulfalizer@gmail.com 2017-09-24T11:47:04Z urn:sha1:b9509c0fbc13679393389611a5c4b88b0324306a This code in zconf.l says !=, not ==. Thought the behavior seemed weird. if (!f && name != NULL && name[0] != '/') { env = getenv(SRCTREE); if (env) { sprintf(fullname, "%s/%s", env, name); f = fopen(fullname, "r"); } } return f; Thankfully only broken for a short while. Also gives much simpler code. 2017-09-24T09:37:42Z Ulf Magnusson ulfalizer@gmail.com 2017-09-24T07:49:20Z urn:sha1:799e6d4e1a4e4a4f59e58298bbb9c01d286159dc Previously, $srctree/path/to/defconfig would be looked up before /path/to/defconfig, and the code wouldn't check if /path/to/defconfig was an absolute path ($srctree is ignored otherwise). Sloppy old oversights. The behavior now fully matches the C implementation. Also fix some related things: - An 'if m' suffices to select a defconfig. We previously required 'y'. - Make the code less hacky and possibly more Windows-friendly by using os.path.relpath() to de-absolutize paths, and stop using os.path.normpath() as it could change the meaning of paths that contain symbolic links. - Explain what happens if 'option defconfig_list' is set on multiple symbols and print a warning in that case. - Fix get_srctree(). It would previously return "." instead of None if $srctree was unset at parse time. Somehow forgot to to test this. The code is now much more straightforward. 2017-09-22T05:02:05Z Ulf Magnusson ulfalizer@gmail.com 2017-09-22T03:59:08Z urn:sha1:8f81cbdcc1d35eeb419c96968d4eedbfeee42e55 - Get rid of _sym_str_string(), which was only used here. - Remove 'if expr is None' case that could never trigger - Add a test for printing string symbols, as they are a bit tricky: Default values should not be evaluated to tristate values. 2017-09-20T03:38:19Z Ulf Magnusson ulfalizer@gmail.com 2017-09-19T21:03:22Z urn:sha1:c8df7316d1ed151b93566a61ed9d95da67e17974 Was added upstream in 31847b67 (kconfig: allow use of relations other than (in)equality). Completely unused (and undocumented) in the kernel except for in DEBUG_UART_8250_WORD in arch/arm/Kconfig.debug: depends on DEBUG_UART_8250_SHIFT >= 2 (That line was added before lt/gt support by the way, and assumed a feature that wasn't there.) This change (and the upstream one) also slightly changes how (in)equality comparisons work, making e.g. MY_HEX = 0x00037 evaluate to 'y' if MY_HEX is 0x37. Prior to this change, the strings needed to match exactly. 2017-09-19T05:29:33Z Ulf Magnusson ulfalizer@gmail.com 2017-09-19T05:06:17Z urn:sha1:4fce6c53e1bdac803c77f0be8bedece22d73c30c Previously, only WEIRD_1 and not WEIRD_2 would be considered not a choice symbol in the following fragment. This lead to a weird warning in U-Boot. choice config FOO config WEIRD_1 depends on FOO if FOO config WEIRD_2 endif endchoice Also add some testcases for weird choice symbols. 2017-09-18T04:49:44Z Ulf Magnusson ulfalizer@gmail.com 2017-09-17T02:23:40Z urn:sha1:fffd30e6ea060b2abebb7ba9cdf881da00ca4cb9 This is like a 'select' that only changes the default value of a symbol, not limiting what values the user can set it to (with one exception: A symbol implied to 'y' can't be set to 'm'). Symbol.get_implied_symbols() was added, corresponding to Symbol.get_selected_symbols(), and Symbol.__str__() was extended to print implied symbols and weak reverse dependencies. Weak reverse dependencies are the 'imply' version of 'select's reverse dependencies. 2015-07-23T19:54:12Z Ulf Magnusson ulfalizer@gmail.com 2015-07-23T19:54:12Z urn:sha1:e15304917c21fbc1d83b02e6ca2e9eb0c8f1963c Mostly to get coverage for _FileFeed.peek_next(), which is only used while reading the header. The tested behavior probably doesn't make much sense for .config files and stems from _FileFeed reuse. It's fairly obscure at least. 2015-06-15T21:29:09Z Ulf Magnusson ulfalizer@gmail.com 2015-06-15T20:10:19Z urn:sha1:0cebc87848210bf99993527694bb96efe45b9598 The line number was previously for logical lines only. Oversight. Get rid of _get_lines() and keep the raw lines in _FileFeed instead, only joining lines with continuation lines as they are fetched. This makes the index correspond to the correct line number from the file. (It also means most lines are returned as-is without any logic applied to them, which is nice.) Litter tests/Klocation with continuation lines to get test coverage. Remove some unused functions that were previously inherited from _Feed and remove it as a base class of _FileFeed. 2015-06-09T14:46:15Z Ulf Magnusson ulfalizer@gmail.com 2015-06-09T14:42:46Z urn:sha1:1f31be32cd2a0c04534c225878f4ce9b7401403f The format of the strings isn't set in stone, but it's nice to verify that nothing changes. This code was incredibly boring to write. :P 2015-06-09T12:28:28Z Ulf Magnusson ulfalizer@gmail.com 2015-06-09T12:28:28Z urn:sha1:0f4882ba3d329005d80b4426688b231381eba5c2