From 92109c88d78d1e84503a94aba329a01fb36da322 Mon Sep 17 00:00:00 2001
From: Michal Berger <michal.berger@intel.com>
Date: Wed, 30 Nov 2022 16:31:19 +0100
Subject: [PATCH] autobuild: Move make_install test into a separate function

Signed-off-by: Michal Berger <michal.berger@intel.com>
Change-Id: Ife9e941b8afb3d33155e336b06c82b52bb4ac292
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15721
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: Kamil Godzwon <kamilx.godzwon@intel.com>
Reviewed-by: Karol Latecki <karol.latecki@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Krzysztof Karas <krzysztof.karas@intel.com>
---
 test/common/autobuild_common.sh | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/test/common/autobuild_common.sh b/test/common/autobuild_common.sh
index 0f59405c4..f81004bf0 100755
--- a/test/common/autobuild_common.sh
+++ b/test/common/autobuild_common.sh
@@ -268,6 +268,10 @@ header_dependency_check() {
 	fi
 }
 
+test_make_install() {
+	$MAKE $MAKEFLAGS install DESTDIR="$SPDK_WORKSPACE" prefix=/usr
+}
+
 test_make_uninstall() {
 	# Create empty file to check if it is not deleted by target uninstall
 	touch "$SPDK_WORKSPACE/usr/lib/sample_xyz.a"
@@ -332,7 +336,7 @@ autobuild_test_suite() {
 		$MAKE $MAKEFLAGS
 		run_test "autobuild_generated_files_check" porcelain_check
 		run_test "autobuild_header_dependency_check" header_dependency_check
-		run_test "autobuild_make_install" $MAKE $MAKEFLAGS install DESTDIR="$SPDK_WORKSPACE" prefix=/usr
+		run_test "autobuild_make_install" test_make_install
 		run_test "autobuild_make_uninstall" test_make_uninstall
 		run_test "autobuild_build_doc" build_doc
 	fi