autobuild: move porcelain check under run_test
Change-Id: I1731e7806b1bf380656b78c1b0fd51711fe3cc68 Signed-off-by: Seth Howell <seth.howell@intel.com> Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/478547 Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com> Reviewed-by: Karol Latecki <karol.latecki@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
This commit is contained in:
parent
ee156e0a11
commit
ab47796eca
17
autobuild.sh
17
autobuild.sh
@ -54,6 +54,14 @@ function make_fail_cleanup {
|
|||||||
false
|
false
|
||||||
}
|
}
|
||||||
|
|
||||||
|
function porcelain_check {
|
||||||
|
if [ $(git status --porcelain --ignore-submodules | wc -l) -ne 0 ]; then
|
||||||
|
echo "Generated files missing from .gitignore:"
|
||||||
|
git status --porcelain --ignore-submodules
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
}
|
||||||
|
|
||||||
if [ $SPDK_RUN_VALGRIND -eq 1 ]; then
|
if [ $SPDK_RUN_VALGRIND -eq 1 ]; then
|
||||||
run_test "valgrind" echo "using valgrind"
|
run_test "valgrind" echo "using valgrind"
|
||||||
fi
|
fi
|
||||||
@ -87,14 +95,7 @@ else
|
|||||||
run_test "make" $MAKE $MAKEFLAGS
|
run_test "make" $MAKE $MAKEFLAGS
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# Check for generated files that are not listed in .gitignore
|
run_test "autobuild_generated_files_check" porcelain_check
|
||||||
timing_enter generated_files_check
|
|
||||||
if [ $(git status --porcelain --ignore-submodules | wc -l) -ne 0 ]; then
|
|
||||||
echo "Generated files missing from .gitignore:"
|
|
||||||
git status --porcelain --ignore-submodules
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
timing_exit generated_files_check
|
|
||||||
|
|
||||||
# Check that header file dependencies are working correctly by
|
# Check that header file dependencies are working correctly by
|
||||||
# capturing a binary's stat data before and after touching a
|
# capturing a binary's stat data before and after touching a
|
||||||
|
Loading…
Reference in New Issue
Block a user