[dpdk-dev] [PATCH] mk: optimize directory dependencies
Robin Jarry
robin.jarry at 6wind.com
Fri Mar 17 18:47:20 CET 2017
Hi Olivier,
Thanks again for this. I only have minor cosmetic remarks:
Le 17 mars 2017 18:13, "Olivier Matz" <olivier.matz at 6wind.com> a écrit:
> diff --git a/app/pdump/Makefile b/app/pdump/Makefile
> index 536198f..8ec6bd6 100644
> --- a/app/pdump/Makefile
> +++ b/app/pdump/Makefile
> @@ -42,7 +42,6 @@ CFLAGS += $(WERROR_FLAGS)
> SRCS-y := main.c
>
> # this application needs libraries first
> -DEPDIRS-y += lib
>
> include $(RTE_SDK)/mk/rte.app.mk
Maybe remove the comment line too?
> diff --git a/app/test-pmd/Makefile b/app/test-pmd/Makefile
> index eec1ed0..3be1ae8 100644
> --- a/app/test-pmd/Makefile
> +++ b/app/test-pmd/Makefile
> @@ -82,7 +82,6 @@ endif
> CFLAGS_cmdline.o := -D_GNU_SOURCE
>
> # this application needs libraries first
> -DEPDIRS-y += lib drivers
>
> include $(RTE_SDK)/mk/rte.app.mk
Same here. There may be others.
> --- a/mk/rte.subdir.mk
> +++ b/mk/rte.subdir.mk
> @@ -59,8 +61,9 @@ build: _postbuild
> .PHONY: $(DIRS-y)
> $(DIRS-y):
> @[ -d $(CURDIR)/$@ ] || mkdir -p $(CURDIR)/$@
> - @echo "== Build $S/$@"
> + @echo "== Build $S/$@ "
Why change this line ?
Robin
More information about the dev
mailing list