Commit b41f404d authored by Jonathan Richardson's avatar Jonathan Richardson Committed by Jim Harris
Browse files

make: Fix python command in Makefile 



Remove detection of python command. If PYTHON_COMMAND isn't set, PYCMD
defaults to python which doesn't exist in some python3 only environments
such as yocto. genconfig.py has already been updated to deprecate
python2 support.

Change-Id: I1a289bc0e30a2de9396e1397a72d8f0e81912a68
Signed-off-by: default avatarJonathan Richardson <jonathan.richardson@broadcom.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/483418


Reviewed-by: default avatarSeth Howell <seth.howell@intel.com>
Reviewed-by: default avatarScott Branden <sbranden@gmail.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Community-CI: Broadcom SPDK FC-NVMe CI <spdk-ci.pdl@broadcom.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 16cc6464

Makefile

+1 −3
Original line number Diff line number Diff line
@@ -103,11 +103,9 @@ mk/cc.mk: 
	false



include/spdk/config.h: mk/config.mk scripts/genconfig.py

	$(Q)PYCMD=$$(cat PYTHON_COMMAND 2>/dev/null) ; \

	test -z "$$PYCMD" && PYCMD=python ; \

	echo "#ifndef SPDK_CONFIG_H" > $@.tmp; \

	echo "#define SPDK_CONFIG_H" >> $@.tmp; \

	$$PYCMD scripts/genconfig.py $(MAKEFLAGS) >> $@.tmp; \

	scripts/genconfig.py $(MAKEFLAGS) >> $@.tmp; \

	echo "#endif /* SPDK_CONFIG_H */" >> $@.tmp; \

	cmp -s $@.tmp $@ || mv $@.tmp $@ ; \

	rm -f $@.tmp