From 49d460555823a701ccb1a079dee35a98cb748c94 Mon Sep 17 00:00:00 2001 From: Michal Berger Date: Mon, 13 Sep 2021 16:00:45 +0200 Subject: [PATCH] scripts/pkgdep: Disable mdl installation on centos Versions of centos we support - 7 and 8 - don't ship with ruby which can handle installation of the mdl gem due to unmet dependencies - it's simply too old. Signed-off-by: Michal Berger Change-Id: Id0a3843bcab12b8de332d1d3f64956f043c347e3 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9483 Community-CI: Mellanox Build Bot Community-CI: Broadcom CI Tested-by: SPDK CI Jenkins Reviewed-by: Ben Walker Reviewed-by: Jim Harris Reviewed-by: Monica Kenguva Reviewed-by: Karol Latecki --- scripts/pkgdep/common.sh | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/scripts/pkgdep/common.sh b/scripts/pkgdep/common.sh index 2c4f05470..55492b3eb 100755 --- a/scripts/pkgdep/common.sh +++ b/scripts/pkgdep/common.sh @@ -109,7 +109,11 @@ install_markdownlint() { if [[ $INSTALL_DEV_TOOLS == true ]]; then install_shfmt install_spdk_bash_completion - install_markdownlint + if [[ $ID != centos ]]; then + install_markdownlint + else + echo "mdl not supported on $ID, disabling" + fi fi if [[ $INSTALL_LIBURING == true ]]; then