From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gavin Hu Subject: [PATCH v8 4/6] devtools: fix the missing ninja command error on CentOS Date: Fri, 15 Jun 2018 16:01:18 +0800 Message-ID: <20180615080120.29253-5-gavin.hu@arm.com> References: <20180615070811.18686-1-gavin.hu@arm.com> <20180615080120.29253-1-gavin.hu@arm.com> Cc: nd@arm.com, gavin.hu@arm.com, stable@dpdk.org To: dev@dpdk.org Return-path: In-Reply-To: <20180615080120.29253-1-gavin.hu@arm.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On CentOS, the ninja executable has a different name: ninja-build, this patch is to fix the missing command error on CentOS as follows: ./devtools/test-meson-builds.sh: line 24: ninja: command not found Fixes: a55277a788 ("devtools: add test script for meson builds") Cc: stable@dpdk.org Signed-off-by: Gavin Hu Reviewed-by: Phil Yang Reviewed-by: Song Zhu --- devtools/test-meson-builds.sh | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh index f2ad05f32..8447c704b 100755 --- a/devtools/test-meson-builds.sh +++ b/devtools/test-meson-builds.sh @@ -10,6 +10,15 @@ srcdir=$(dirname $(readlink -m $0))/.. MESON=${MESON:-meson} +if command -v ninja >/dev/null 2>&1 ; then + ninja_cmd=ninja +elif command -v ninja-build >/dev/null 2>&1 ; then + ninja_cmd=ninja-build +else + echo "## ERROR: ninja is missing, please install..." + exit 1 +fi + build () # { builddir=$1 @@ -21,7 +30,7 @@ build () # unset CC fi echo "ninja -C $builddir" - ninja -C $builddir + $ninja_cmd -C $builddir } # shared and static linked builds with gcc and clang -- 2.11.0