Kconfiglib.git, branch v2.6.0Fork of https://github.com/zephyrproject-rtos/Kconfiglib.githttps://git.mcdonnell.dev/Kconfiglib.git/atom?h=v2.6.02018-01-30T10:10:51ZMake 2.6.0 release2018-01-30T10:10:51ZUlf Magnussonulfalizer@gmail.com2018-01-30T10:10:51Zurn:sha1:1e40733b7caff0a5f84eee6775e851e1d32f95a9
This release tightens up syntax checking and adds all the warnings
generated by the C implementation (plus a few extra ones), making
Kconfiglib more viable as a replacement for the C tools, rather than
just being an auxiliary library.
Syntax checking changes:
- a84848b ("Tighten up syntax checking")
- 8716316 ("Generate a parse error for extra tokens at EOL")
- 416083a ("Flag constant symbols where they're not allowed")
Warning changes:
- 9c309400 ("Add some post-parsing warnings")
- da0bfa3a ("Error out for malformed hex/int/string defaults")
- 5d2041a9 ("Add more choice type and prompt sanity checks")
- 67f825fc ("Warn if there's more than one help text")
- 990c780f ("Add some warnings related to selects and implies")
- 7432ade8 ("Warn if a symbol is defined with multiple types")
- 5eeea863 ("Improve int/hex sanity checking")
- ad94bb45 ("Sanity-check range values")
- e930c140 ("Warn if a symbol/choice has multiple prompts")
- 60d12911 ("Warn if help text is empty")
- d43d2e83 ("Warn if menuconfig statement has no prompt")
- 3db12f72 ("Warn if a choice symbol has defaults")
- 0eadd61c ("Warn if choice symbol has prompt outside choice")
Also includes various code nits.
Fix warnings printed for test suite2018-01-30T09:58:24ZUlf Magnussonulfalizer@gmail.com2018-01-30T09:58:24Zurn:sha1:ef627e02c8c3f5423e25e2a53eb61889d610ca60
Just to have clean output. The warnings themselves are accurate.
Warn if choice symbol has prompt outside choice2018-01-30T09:46:38ZUlf Magnussonulfalizer@gmail.com2018-01-30T09:43:04Zurn:sha1:0eadd61c05c523a150721f9646e4d3f7cff58e5c
Defining a choice symbol in multiple places to add some properties to it
outside the choice seems to actually be done deliberately by MIPS, but
it's almost guaranteed to be an error if the definition(s) outside the
choice have a prompt (and so can be changed by the user there), so warn
for that.
Warn if a choice symbol has defaults2018-01-29T13:43:49ZUlf Magnussonulfalizer@gmail.com2018-01-29T13:43:31Zurn:sha1:3db12f728ec2d48640574133675bb01ca926d741
Never has en affect.
Warn if menuconfig statement has no prompt2018-01-29T13:26:01ZUlf Magnussonulfalizer@gmail.com2018-01-29T12:58:38Zurn:sha1:d43d2e83a9dc96d86dfbc946516bc3090303dbd3
Mirrors a warning in the C implementation.
Give tip about getting rid of warnings2018-01-29T12:42:26ZUlf Magnussonulfalizer@gmail.com2018-01-29T12:42:26Zurn:sha1:097e7756caae0c87ab7891f7fb84e7213f59a595
The empty-help warning is pretty spammy at the moment.Warn if help text is empty2018-01-29T12:36:41ZUlf Magnussonulfalizer@gmail.com2018-01-29T12:34:49Zurn:sha1:60d129111d8195eb61a010994e9fc974365a4d63
Personal pet peeve. Should add a warning to the C implementation too.
Refactor _check_sym_sanity()2018-01-29T10:56:28ZUlf Magnussonulfalizer@gmail.com2018-01-29T10:55:04Zurn:sha1:797221c4e39fbc584ba67c1bbecff383441923ff
Arrange by type, which works pretty neatly except for the pesky ranges
check. This also makes us do less work in the common BOOL/TRISTATE case.
Get rid of _check_select_imply_sanity().
Move sym.ranges check to end of _check_sym_sanity()2018-01-29T10:26:22ZUlf Magnussonulfalizer@gmail.com2018-01-29T10:26:22Zurn:sha1:92b1fcdd4bed1c3ee1bdbac5dccbb4f4baa447b7
More important stuff up front.
Warn if a symbol/choice has multiple prompts2018-01-29T09:48:45ZUlf Magnussonulfalizer@gmail.com2018-01-29T09:46:57Zurn:sha1:e930c140b424e4f72c6317bedef4ab45079270c9
In a single location. Having multiple definitions with different prompts
is okay.