Commit 157b1a33 authored by Alexey Marchuk's avatar Alexey Marchuk Committed by Tomasz Zawadzki
Browse files

configure: Check nasm - redirect stderr to null device 



configure prints an error if nasm is not installed on the system:
./configure: line 554: nasm: command not found
Notice: ISA-L, compression & crypto auto-disabled due to nasm dependency.
These features require NASM version 2.13.03 or newer.  Please install
or upgrade then re-run this script.
Redirect stderr to null device to disable error printing

Change-Id: Ifa596fba19ecaf5270614ab22f09f0a66af08475
Signed-off-by: default avatarAlexey Marchuk <alexeymar@mellanox.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/871


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
parent 5c9c24e1

configure

+1 −1
Original line number Diff line number Diff line
@@ -551,7 +551,7 @@ if [[ "${CONFIG[FC]}" = "y" ]]; then 
fi



if [[ "${CONFIG[ISAL]}" = "y" ]] || [[ "${CONFIG[CRYPTO]}" = "y" ]]; then

	ver=$(nasm -v | awk '{print $3}' | sed 's/[^0-9]*//g')

	ver=$(nasm -v 2>/dev/null | awk '{print $3}' | sed 's/[^0-9]*//g')

	if [[ "${ver:0:1}" -le "2" ]] && [[ "${ver:0:3}" -le "213" ]] && [[ "${ver:0:5}" -lt "21303" ]]; then

		echo "Notice: ISA-L, compression & crypto auto-disabled due to nasm dependency."

		echo "These features require NASM version 2.13.03 or newer.  Please install"