Fork of https://github.com/zephyrproject-rtos/Kconfiglib.git https://git.mcdonnell.dev/Kconfiglib.git/atom?h=v4.0.0 2018-03-11T07:11:56Z 2018-03-11T07:11:56Z Ulf Magnusson ulfalizer@gmail.com 2018-03-11T07:11:56Z urn:sha1:44f782723b71e4eb4661e61cc6f887aefe00898e Includes a small API behavior change, hence the major version bump: - db63301 ("Return "" for unwritten symbols in Symbol.config_string") New features: - Support for incremental builds using the same scheme as the include/config/ directory in the C tools. See the new Kconfig.sync_deps() function and commit 378dedc ("Add support for incremental builds"). - Minimal configuration (defconfig) writing, generating identical files to the C implementation. See commit 652f11b ("Add minimal configuration file generation support"). The C 'make savedefconfig' implementation is currently broken in obscure cases involving tristate choices (due to broken invalidation code), so the tests are disabled for now. I have a patch for the C implementation that I will submit. Fixes: - user_value wasn't set for choice symbols set to y via load_config() with replace=True. Fixed by commit 0bd841d ("Properly remember y user values for choice symbols"). Didn't affect correctness, only the API. Other changes: - 778ec47 ("Only write '# CONFIG_FOO is not set' for visible symbols") mirrors a change I made to the C tools. - bbe3ae9 ("Do not write the defconfig_list symbol to .config") mirrors another change made to the C tools. - Misc. refactoring and cleanup. 2018-03-11T06:29:19Z Ulf Magnusson ulfalizer@gmail.com 2018-03-05T19:10:01Z urn:sha1:652f11b60f7b738440fe0e919d4de0c188e80395 Works like 'make savedefconfig' in the C tools. Call it write_min_config() rather than write_defconfig() to be a bit more explicit. Add a test similar to test_defconfig that compares Kconfiglib minimal configuration output against the C implementation, for all defconfig files. Disable the tests for now. The C tools have a bug that causes an incorrect configuration to be generated for tristate choices in some cases. They will be re-enabled once those are fixed. 2018-03-11T02:01:50Z Ulf Magnusson ulfalizer@gmail.com 2018-03-11T01:45:14Z urn:sha1:0bd841def7f3911771afa2346acae4938033ace9 Commit e8b4ecb ("Don't special-case user_value for choice symbols set to y") was meant to make user_value reflect an y value assigned to a choice symbol, but didn't get it right for load_config() with replace=True: The user value was set at first, but was then unset again due to a misplaced '_was_set = True' assignment. This commit makes y choice symbol user values always be remembered. The correctness of generated files was not affected, since the user selection of a choice is actually remembered as Choice.user_selection. The previous commit just turned into a no-op for load_config() with replace=True. 2018-03-09T23:01:09Z Ulf Magnusson ulfalizer@gmail.com 2018-03-09T22:58:58Z urn:sha1:02d1131cf6ee76ce89e63121aa90512e153e15bf defconfig_files() yields all the defconfig files for a particular arch/ subdirectory. This will allow reuse when savedefconfig is tested. Also simplify the code a bit. 2018-03-09T22:17:15Z Ulf Magnusson ulfalizer@gmail.com 2018-03-09T22:17:15Z urn:sha1:63306858cf439b6de608b9129e602cf7ea6ddcf0 Not that important of a stat, plus there might be more tests that process defconfig files soon, making it kinda meaningless. 2018-03-06T03:15:44Z Ulf Magnusson ulfalizer@gmail.com 2018-03-06T03:14:22Z urn:sha1:77d1cc9302f2b18cec6763967e53ad8cbcce36c5 No great need to log timestamps anymore. 2018-03-06T00:41:17Z Ulf Magnusson ulfalizer@gmail.com 2018-03-06T00:41:17Z urn:sha1:00d4f53298ccd1a6e0b5f8b9f60df82bc746f29f The 'for' overrode 'srcarch', which is currently harmless, but ugly. 2018-03-06T00:23:29Z Ulf Magnusson ulfalizer@gmail.com 2018-03-06T00:22:20Z urn:sha1:6817b67efeea40288f1ad840141a1c41e03ded9a Makes sense as SRCARCH holds the arch/ subdirectory and _arch might be confused for ARCH. 2018-03-05T23:25:56Z Ulf Magnusson ulfalizer@gmail.com 2018-03-05T23:24:32Z urn:sha1:4070b29473d74ae586f39b984d211ac11a3a164b Get rid of the compare_configs flag and just do the comparison from the test functions themselves with a helper. Also get rid of the test_load() test. That's indirectly tested through the other tests, and test_alldefconfig() runs first and is speedy. 2018-03-05T22:44:48Z Ulf Magnusson ulfalizer@gmail.com 2018-03-05T21:56:53Z urn:sha1:b499d06a8513ba599ca72d4d7bb6ae7fb6f1a9f9 The original idea was that it might be handy to look up docstrings for internal functions from the Python prompt (or an IDE) while figuring out the code. Not sure it's that useful in practice though. Comments shorten the code a bit, and might make it clearer at a glance that the function is internal.