On 09/29/2015 03:00 AM, Mario Carrillo wrote:
> Add hierarchy-file support to the DPDK headers,
> when invoking "make install-headers" headers will
> be installed in: $(DESTDIR)/usr/include/dpdk
> This hierarchy is based on:
> http://www.freedesktop.org/software/systemd/man/file-hierarchy.html
>
> Signed-off-by: Mario Carrillo <mario.alfredo.c.arevalo at intel.com>
> ---
> mk/rte.sdkinstall.mk | 17 +++++++++++++++++
> mk/rte.sdkroot.mk | 4 ++--
> 2 files changed, 19 insertions(+), 2 deletions(-)
>
> diff --git a/mk/rte.sdkinstall.mk b/mk/rte.sdkinstall.mk
> index 86c98a5..3ec344e 100644
> --- a/mk/rte.sdkinstall.mk
> +++ b/mk/rte.sdkinstall.mk
> @@ -40,7 +40,12 @@ endif
> # target name or a name containing jokers "*". Example:
> # x86_64-native-*-gcc
> ifndef T
> +ifeq (,$(wildcard $(BUILD_DIR)/build/.config))
> T=*
> +else
> +INCLUDE_DIR := $(DESTDIR)/usr/include/dpdk
This makes it cumbersome to override INCLUDE_DIR. Please keep $(DESTDIR)
separate and only use it when actually copying, and only set INCLUDE_DIR
here if not already set, ie
INCLUDE_DIR ?= /usr/include/dpdk
...and then something like:
@[ -d $(DESTDIR)/$(INCLUDE_DIR) ] || mkdir -p $(DESTDIR)/$(INCLUDE_DIR)
@for HSLINK in ${HSLINKS}; do \
HEADER=$$(readlink -f $$HSLINK); \
cp -rf $$HEADER ${DESTDIR}/${INCLUDE_DIR}; \
echo installing: $$HEADER; \
The same goes for all the locations in the other patches.
The need to override includedir might not be that common, but for
example it could be used for installing multiple versions in parallel.
For others like LIB_DIR the ability to override is truly required
because distros differ wildly in this regard.
Thanks for working on this!
- Panu -
