On Fri, Sep 29, 2017 at 10:25 PM, Tom Rini <tr...@konsulko.com> wrote: > On Tue, Sep 19, 2017 at 12:52:55PM +0200, Ulf Magnusson wrote: > >> Corresponds to 375506d (File writing nit) from upstream >> (https://github.com/ulfalizer/Kconfiglib). >> >> Adds proper 'imply' support and fixes a few minor issues, one of which >> previously triggered the following weird warning: >> >> configs/taurus_defconfig: /tmp/tmpisI45S:6: warning: assignment to >> SPL_LDSCRIPT changes mode of containing choice from >> "arch/$(ARCH)/cpu/u-boot-spl.lds" to "y" >> >> The change in 8639f69 (genconfig.py: Print defconfig next to warnings) >> was reapplied. >> >> tools/moveconfig.py previously depended on a hack that merged 'select's >> with 'imply's. It was modified to look at the union of >> Symbol.get_selected_symbols() and Symbol.get_implied_symbols(), which >> should give the same behavior. >> >> tools/genboardscfg.py was verified to produce identical board.cfg's >> before and after the change. >> >> Signed-off-by: Ulf Magnusson <ulfali...@gmail.com> > > Applied to u-boot/master, thanks! > > -- > Tom
For future reference, Kconfiglib is now available on PyPI and can be installed via e.g. 'pip' (see the updated README at https://github.com/ulfalizer/Kconfiglib). Maybe the custom patch in 8639f69a (genconfig.py: Print defconfig next to warnings) could be removed if stderr is temporarily redirected to a string (https://stackoverflow.com/questions/1218933/can-i-redirect-the-stdout-in-python-into-some-sort-of-string-buffer). I was thinking of adding an optional list argument to some functions that would receive the warning strings if provided, but I'm not sure if it's worth it. Would want them printed directly by default at least, because otherwise people will miss them. Cheers, Ulf _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
