ktest: Fix kernelrevision with POST_BUILD
authorSteven Rostedt <srostedt@redhat.com>
Fri, 18 May 2012 17:34:35 +0000 (13:34 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Fri, 18 May 2012 18:27:51 +0000 (14:27 -0400)
commit683a3e6481a5cffc58496a590decf65909d0454b
tree0c00e2379b586747c080c92107e2ec7c7f4fa345
parent36be50515fe2aef61533b516fa2576a2c7fe7664
ktest: Fix kernelrevision with POST_BUILD

The PRE_BUILD and POST_BUILD options of ktest are added to allow the
user to add temporary patch to the system and remove it on builds. This
is sometimes use to take a change from another git branch and add it to
a series without the fix so that this series can be tested, when an
unrelated bug exists in the series.

The problem comes when a tagged commit is being used. For example, if
v3.2 is being tested, and we add a patch to it, the kernelrelease for
that commit will be 3.2.0+, but without the patch the version will be
3.2.0. This can cause problems when the kernelrelease is determined for
creating the /lib/modules directory. The kernel booting has the '+' but
the module directory will not, and the modules will be missing for that
boot, and may not allow the kernel to succeed.

The fix is to put the creation of the kernelrelease in the POST_BUILD
logic, before it applies the POST_BUILD operation. The POST_BUILD is
where the patch may be removed, removing the '+' from the kernelrelease.

The calculation of the kernelrelease will also stay in its current
location but will be ignored if it was already calculated previously.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
tools/testing/ktest/ktest.pl