On Fri, Nov 13, 2020 at 2:53 PM Bruce Richardson <bruce.richard...@intel.com> wrote: > > +NEED_CRYPTO_SCHEDULER = $(shell echo RTE_CRYPTO_SCHEDULER | $(CPP) > > $(CFLAGS) -P - | tail -1) > > +ifeq ($(NEED_CRYPTO_SCHEDULER), 1) > > Sorry for the last-minute comment, but I wonder for this check if we can do > better by adding into each makefile something like: > > CONFIG_DEFINES=$(shell $(CC) $(CFLAGS) -dM -E - < /dev/null) > > Then we can easily do multiple checks for vars as needed using findstring, > e.g. > > ifeq ($(findstring RTE_CRYPTO_SCHEDULER,$(CONFIG_DEFINES),) > $(info No crypto scheduler found) > else > ... > endif > > Whatever approach we use here, I'd like applicable across all makefiles for > consistency, and shelling out per-value seems wasteful. Pulling all macro > values also allows checks for architecture and instruction set levels too, > if so desired.
I'll have a try.. -- David Marchand
