Kconfiglib.git/testsuite.py, branch v3.0.0

Don't special-case user_value for choice symbols set to y

2018-02-08T09:14:48Z

Previously, setting a choice symbol to y would only update user_selection on the parent choice and not the symbol's own user_value. Now both are updated. The point of the old behavior was to remember the m mode selections of a choice when it was switched back and forth between m and y mode, which was a feature I thought the C implementation had. On closer inspection, the C implementation never had that feature, though it might appear like it if you only make "lucky" changes (if you never select any symbols in y mode that were n in m mode). The new behavior is simpler and easier to understand: Symbol.user_value now always matches the value assigned in a .config file or via set_value(), provided the value was well-formed. This might avoid some special-casing in scripts too. The loss in usability is pretty minimal.

Add back omitted $ARCH != $SRCARCH defconfig tests

2018-02-07T05:34:22Z

Oversight during refactoring. defconfig files should be searched for in in arch/$SRCARCH/ rather than arch/$ARCH/, to account for $ARCH != $SRCARCH. Adds a few tests to test_defconfig(). Some of the ARCH/defconfig combinations might be nonsensical, but that's fine. Make all test functions take $SRCARCH as well as $ARCH, which makes the implementation straightforward.

Refactor test suite arch Kconfig loading

2018-02-03T18:18:22Z

List the ARCH/SRCARCH pairs that don't conform to the arch/$ARCH/Kconfig pattern directly.

Set $KERNELVERSION properly in test suite

2018-02-03T17:13:22Z

Don't think anything currently depends on it where it would matter for the test suite, but might as well.

Fix warnings printed for test suite

2018-01-30T09:58:24Z

Just to have clean output. The warnings themselves are accurate.

Add some post-parsing warnings

2018-01-28T09:56:57Z

These are easiest to check after parsing, since a symbol/choice can be defined in multiple locations: - Warn if a symbol or choice defined without a type. Also warn for choice value symbols defined without a type, even if they automatically get their type from the choice. This feature isn't well-known and probably not used deliberately. - Warn if a choice is defined without a prompt - Warn of a choice default symbol is not contained in the choice Also move _name_and_loc_str() from the symbol class to the global scope and generalize it to be able to handle choices.

Flag constant symbols where they're not allowed

2018-01-28T05:20:12Z

Might break U-Boot if they ever upgrade Kconfiglib, because they have a 'select y' in there (which the C tools don't flag since they only look for quotes), but it's a one-line fix.

Support <, > relational operators with tristates

2018-01-24T23:18:40Z

Mirrors 9059a3493ef ("kconfig: fix relational operators for bool and tristate symbols") in the C implementation.

Simplify escape()

2018-01-22T19:04:41Z

The fancy regex isn't really justified. Much faster with replace() too, though this is an unlikely hotspot. Could have used a \g<0> backreference to refer to the entire match instead of using a capturing group too. Hadn't discovered that. Add some selftests for escape() and unescape() too.

Whitespace nits

2018-01-21T16:37:06Z