Kconfiglib.git/kconfiglib.py, branch v10.33.0Fork of https://github.com/zephyrproject-rtos/Kconfiglib.githttps://git.mcdonnell.dev/Kconfiglib.git/atom?h=v10.33.02018-12-15T13:03:14ZFix user-defined preprocessor function example code2018-12-15T13:03:14ZUlf Magnussonulfalizer@gmail.com2018-12-15T13:03:14Zurn:sha1:a88302bcfbf73f9b8b50bea593beb82534bfcc63
The functions must be defined before they can be put into 'functions'.
Improve error for missing endchoice/endif/endmenu2018-12-15T12:54:34ZUlf Magnussonulfalizer@gmail.com2018-12-15T12:54:34Zurn:sha1:02669243c44615852ab8b87c62ff0bdfd0e666dd
Say what was expected in the error message.
Store file.readline in _readline instead of having _file2018-12-15T12:48:48ZUlf Magnussonulfalizer@gmail.com2018-12-15T12:04:08Zurn:sha1:d4929c497404fd1761d8351455a67e748c381d53
Gets rid of some lookups.
A bigger hotspot is all the _next_line() calls though. Wonder if there's
some nice restructuring to avoid them. Getting rid of the _reuse_tokens
flag would be nice too.
Tighten up _next_line()2018-12-15T11:43:55ZUlf Magnussonulfalizer@gmail.com2018-12-15T11:43:55Zurn:sha1:3316fb3d9f37fcae12c5ef5abe008daa772202c1
self._line is only used for error reporting, and the empty string
returned at EOF would never be shown. Move the assignment to after
readline() and use a local variable to hold the line instead, to get rid
of some property lookups.
Check isspace() before EOF in _parse_help()2018-12-15T11:33:25ZUlf Magnussonulfalizer@gmail.com2018-12-15T11:32:45Zurn:sha1:e34eb67a736c765cd1f858de80ff50157eda18bc
More common. isspace() returns False for empty strings.
Clarify that load_config()'s filename argument defaults to None2018-12-09T02:18:32ZUlf Magnussonulfalizer@gmail.com2018-12-09T01:54:49Zurn:sha1:be3c6827883d3726beb2b8d361987f0efb0e8950
Had missed the '(default: None)', and it doesn't hurt to point it out in
the description either.
Point out that filename=None is the default in the write_config()
'filename' description too, though it already had the '(default: None)'.
_save_old() code nits2018-12-08T13:13:42ZUlf Magnussonulfalizer@gmail.com2018-12-08T13:13:42Zurn:sha1:094f4a9622046924032c80768a9760e20f738f80Simplify 'verbose' check in write_config()2018-12-08T13:02:52ZUlf Magnussonulfalizer@gmail.com2018-12-08T13:02:00Zurn:sha1:54c58735472f7a03c0f9370572ce1c778da64becPreserve symlinks when writing .config.old files2018-12-08T03:54:47ZUlf Magnussonulfalizer@gmail.com2018-12-08T03:47:52Zurn:sha1:78073d6674ce3250334d0f5177c9f8010e8d381c
if .config is a symlink, then the intention is probably to overwrite the
target, but rename()ing the symlink to .config.old interferes with that.
Use the shutil.copyfile() fallback instead if .config is a symlink.
Make {load,write}_config(filename=None) implement the standard behavior2018-12-08T02:28:52ZUlf Magnussonulfalizer@gmail.com2018-12-07T23:41:01Zurn:sha1:7b4f72fa7a19b7c38312a6f6fd553ca45f04e9a6
Make the previously obligatory 'filename' argument to load_config() and
write_config() default to None, and have that implement the behavior
you'd usually want: read/write either KCONFIG_CONFIG or ".config" if
unset, and read the 'option defconfig_list' configuration file if
KCONFIG_CONFIG/".config" doesn't exist.
For load_config(), filename=None also allows the configuration file to
be missing without raising an error. load_config() returns True if a
local configuration file was loaded, which is useful to check in the
menuconfig (if no local configuration file exists, we always want to
prompt for saving the configuration when exiting).
Also add a 'verbose' argument (default True) to load_config() and
read_config() that makes them print which files were read/written in
filename=None mode.
Also generalize olddefconfig.py and oldconfig.py to not require there to
already be a local configuration file. This was a bit silly for
olddefconfig.py in particular.
Remove the examples/defconfig.py script. It's a duplicate of
olddefconfig.py.