Fork of https://github.com/zephyrproject-rtos/Kconfiglib.git https://git.mcdonnell.dev/Kconfiglib.git/atom?h=v1.0.0 2017-09-22T05:02:05Z 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 2015-06-07T15:46:11Z Ulf Magnusson ulfalizer@gmail.com 2015-06-07T14:57:46Z urn:sha1:d56e9c1d4c3e6f560382ac71fa2419b2a9bef1cb (It was never required if you explicitly passed a 'base_dir', but it's a bit silly to have to do that too.) This is a bug. I expected os.path.expandvars() to replace references non-set environment variables with nothing, but it leaves them as is. Work around it by letting base_dir = None be special and the default. It uses $srctree if set and the current directory otherwise. This has the following advantages: - It avoids having to reimplement a different version of os.path.expandvars() and special-casing "" to mean the current directory. - It means '$' can appear in paths. (Though it probably never will.) Maybe the expansion behavior could be removed too, but keep it for now to be backwards compatible. 2015-06-06T06:37:57Z Ulf Magnusson ulfalizer@gmail.com 2015-06-06T06:37:57Z urn:sha1:a73f978e4f8e032c2c2668e9c6d1b84840cc619c