build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48679 )
Change subject: util: pull in Kconfiglib ......................................................................
Patch Set 3:
(36 comments)
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst File util/Kconfiglib/README.rst:
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 208: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 214: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 327: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 367: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 385: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 405: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 449: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 456: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 461: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 469: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 477: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 485: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 507: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 512: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 525: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 539: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 757: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 766: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/README.rst@... PS3, Line 817: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/kconfiglib.... File util/Kconfiglib/kconfiglib.py:
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/kconfiglib.... PS3, Line 777: was created, or the empty string if $srctree wasn't set. This gives nice 'wasn' may be misspelled - perhaps 'was'?
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/kconfiglib.... PS3, Line 793: was created, or "CONFIG_" if CONFIG_ wasn't set. This is the prefix used 'wasn' may be misspelled - perhaps 'was'?
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/kconfiglib.... PS3, Line 800: KCONFIG_CONFIG_HEADER wasn't set. This string is inserted verbatim at the 'wasn' may be misspelled - perhaps 'was'?
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/kconfiglib.... PS3, Line 806: KCONFIG_AUTOHEADER_HEADER wasn't set. This string is inserted verbatim at 'wasn' may be misspelled - perhaps 'was'?
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/kconfiglib.... PS3, Line 3253: "wasn't used.", 'wasn' may be misspelled - perhaps 'was'?
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/kconfiglib.... PS3, Line 4354: # contraints, it is stored in exactly the same form as 'contraints' may be misspelled - perhaps 'constraints'?
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/makefile.pa... File util/Kconfiglib/makefile.patch:
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/makefile.pa... PS3, Line 15: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/makefile.pa... PS3, Line 46: -- trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/menuconfig.... File util/Kconfiglib/menuconfig.py:
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/menuconfig.... PS3, Line 1177: # show-all because the first entry wasn't visible, try turning it off. 'wasn' may be misspelled - perhaps 'was'?
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/tests/Kprep... File util/Kconfiglib/tests/Kpreprocess:
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/tests/Kprep... PS3, Line 107: $(undefined) trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/tests/Krang... File util/Kconfiglib/tests/Krange:
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/tests/Krang... PS3, Line 73: hex "hex range 10-20" trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/tests/Krang... PS3, Line 81: int "int range 10-20" trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/testsuite.p... File util/Kconfiglib/testsuite.py:
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/testsuite.p... PS3, Line 22: # Like obsessive, for the minimal configuation (defconfig) tests. 'configuation' may be misspelled - perhaps 'configuration'?
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/testsuite.p... PS3, Line 166: "the assigned user value '{}' wasn't reflected in user_value " 'wasn' may be misspelled - perhaps 'was'?
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/testsuite.p... PS3, Line 512: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/testsuite.p... PS3, Line 514: trailing whitespace
https://review.coreboot.org/c/coreboot/+/48679/3/util/Kconfiglib/testsuite.p... PS3, Line 2731: # Runs tests on configurations from the kernel. Tests compability with the 'compability' may be misspelled - perhaps 'compatibility'?