[dpdk-dev] [PATCH v2 1/8] mk: Add rule for installing headers
Panu Matilainen
pmatilai at redhat.com
Tue Sep 29 10:49:18 CEST 2015
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 -
More information about the dev
mailing list