diff mbox

linux-gen: abi: add symlink to arch specific dir

Message ID 1487245689-4169-1-git-send-email-petri.savolainen@linaro.org
State Accepted
Commit 52f32ce1fa6b5416c59ead45ff27d731d98b8f87
Headers show

Commit Message

Petri Savolainen Feb. 16, 2017, 11:48 a.m. UTC
Added symbolic link to the installed include directory. The link
points to the arch specific abi dir, so that application does
not need to add the arch specific dir into its makefile.

Signed-off-by: Petri Savolainen <petri.savolainen@linaro.org>

---
 configure.ac                       | 2 ++
 platform/linux-generic/Makefile.am | 6 ++++++
 2 files changed, 8 insertions(+)

-- 
2.8.1

Comments

Maxim Uvarov Feb. 16, 2017, 8:31 p.m. UTC | #1
that also fixes compilation for apps against installed odp, right?

Maxim.

On 02/16/17 14:48, Petri Savolainen wrote:
> Added symbolic link to the installed include directory. The link

> points to the arch specific abi dir, so that application does

> not need to add the arch specific dir into its makefile.

> 

> Signed-off-by: Petri Savolainen <petri.savolainen@linaro.org>

> ---

>  configure.ac                       | 2 ++

>  platform/linux-generic/Makefile.am | 6 ++++++

>  2 files changed, 8 insertions(+)

> 

> diff --git a/configure.ac b/configure.ac

> index 6153efd..b8e34fe 100644

> --- a/configure.ac

> +++ b/configure.ac

> @@ -53,6 +53,8 @@ AC_PROG_CXX

>  AC_PROG_INSTALL

>  AC_PROG_MAKE_SET

>  

> +AC_PROG_LN_S

> +

>  AM_PROG_AR

>  #Use libtool

>  LT_INIT([])

> diff --git a/platform/linux-generic/Makefile.am b/platform/linux-generic/Makefile.am

> index 576fedf..092487c 100644

> --- a/platform/linux-generic/Makefile.am

> +++ b/platform/linux-generic/Makefile.am

> @@ -212,3 +212,9 @@ __LIB__libodp_linux_la_SOURCES = \

>  if HAVE_PCAP

>  __LIB__libodp_linux_la_SOURCES += pktio/pcap.c

>  endif

> +

> +# Create symlink for ABI header files. Application does not need to use the arch

> +# specific include path for installed files.

> +install-data-hook:

> +	$(LN_S) -r $(prefix)/include/odp/arch/@ARCH_ABI@/odp/api/abi \

> +$(prefix)/include/odp/api/abi

>
Maxim Uvarov Feb. 17, 2017, 7:47 p.m. UTC | #2
tested and merged this patch.

On 02/16/17 14:48, Petri Savolainen wrote:
> Added symbolic link to the installed include directory. The link

> points to the arch specific abi dir, so that application does

> not need to add the arch specific dir into its makefile.

> 

> Signed-off-by: Petri Savolainen <petri.savolainen@linaro.org>

> ---

>  configure.ac                       | 2 ++

>  platform/linux-generic/Makefile.am | 6 ++++++

>  2 files changed, 8 insertions(+)

> 

> diff --git a/configure.ac b/configure.ac

> index 6153efd..b8e34fe 100644

> --- a/configure.ac

> +++ b/configure.ac

> @@ -53,6 +53,8 @@ AC_PROG_CXX

>  AC_PROG_INSTALL

>  AC_PROG_MAKE_SET

>  

> +AC_PROG_LN_S

> +

>  AM_PROG_AR

>  #Use libtool

>  LT_INIT([])

> diff --git a/platform/linux-generic/Makefile.am b/platform/linux-generic/Makefile.am

> index 576fedf..092487c 100644

> --- a/platform/linux-generic/Makefile.am

> +++ b/platform/linux-generic/Makefile.am

> @@ -212,3 +212,9 @@ __LIB__libodp_linux_la_SOURCES = \

>  if HAVE_PCAP

>  __LIB__libodp_linux_la_SOURCES += pktio/pcap.c

>  endif

> +

> +# Create symlink for ABI header files. Application does not need to use the arch

> +# specific include path for installed files.

> +install-data-hook:

> +	$(LN_S) -r $(prefix)/include/odp/arch/@ARCH_ABI@/odp/api/abi \

> +$(prefix)/include/odp/api/abi

>


added 2 tabs here for better view.

Maxim.
diff mbox

Patch

diff --git a/configure.ac b/configure.ac
index 6153efd..b8e34fe 100644
--- a/configure.ac
+++ b/configure.ac
@@ -53,6 +53,8 @@  AC_PROG_CXX
 AC_PROG_INSTALL
 AC_PROG_MAKE_SET
 
+AC_PROG_LN_S
+
 AM_PROG_AR
 #Use libtool
 LT_INIT([])
diff --git a/platform/linux-generic/Makefile.am b/platform/linux-generic/Makefile.am
index 576fedf..092487c 100644
--- a/platform/linux-generic/Makefile.am
+++ b/platform/linux-generic/Makefile.am
@@ -212,3 +212,9 @@  __LIB__libodp_linux_la_SOURCES = \
 if HAVE_PCAP
 __LIB__libodp_linux_la_SOURCES += pktio/pcap.c
 endif
+
+# Create symlink for ABI header files. Application does not need to use the arch
+# specific include path for installed files.
+install-data-hook:
+	$(LN_S) -r $(prefix)/include/odp/arch/@ARCH_ABI@/odp/api/abi \
+$(prefix)/include/odp/api/abi