Hi Thomas, On 06/22/2016 01:49 PM, Thomas Monjalon wrote: > 2016-06-22 14:34, Panu Matilainen: >> --- a/lib/librte_port/Makefile >> +++ b/lib/librte_port/Makefile >> @@ -82,6 +82,8 @@ DEPDIRS-$(CONFIG_RTE_LIBRTE_PORT) += lib/librte_mempool >> DEPDIRS-$(CONFIG_RTE_LIBRTE_PORT) += lib/librte_ether >> DEPDIRS-$(CONFIG_RTE_LIBRTE_PORT) += lib/librte_ip_frag >> DEPDIRS-$(CONFIG_RTE_LIBRTE_PORT) += lib/librte_sched >> +ifeq ($(CONFIG_RTE_LIBRTE_KNI),y) >> DEPDIRS-$(CONFIG_RTE_LIBRTE_PORT) += lib/librte_kni >> +endif > > I do not remember why $(CONFIG_RTE_LIBRTE_PORT) is needed in its Makefile. > I think we can do > DEPDIRS-$(CONFIG_RTE_LIBRTE_KNI) += lib/librte_kni > and set DEPDIRS-y everywhere else. >
It's probably not much used, but the build framework allows to do the following to build only one directory: make lib/librte_port_sub This directly jumps to the librte_port Makefile, bypassing parent directories. I think that's why the config check is duplicated in the Makefile. Olivier
