On Tue, 28 Jun 2016 13:19:06 +0300, Alexander Kapshuk said: > Definition of the 'printversion()' function. The function tests whether > the variable that contains the version number is empty, and prints > the name of the utility and its version number as a formatted string, > if the version number is not an empty value.
This needs to be the first patch in the series, not the last, so that if you're applying the patches one by one, the result still works, which allows incremental testing after each patch. Putting it last means you have to apply all 32 patches before you get something you can test. The idea is good, however. One thing that might be good now that it's only one chunk of code, is to add some code to check between the following cases: it's something like isdnctrl or cardmgr that's not installed because it's truly optional in today's world where ISDN or PCMCIA slots have become rare, or if the regexp doing the matching failed because the utility is present but produced unexpected output. Another useful thing would be distinguishing between must-have things like the toolchain where the build *will* fail, and optionals that are only used in some configurations. This will probably require reordering the output (and corresponding changes to Documentation/Changes)
pgpCWOdjPfzR0.pgp
Description: PGP signature
