[PATCH] libffi: Copy headers to more common include path.

[PATCH] libffi: Copy headers to more common include path.

[Fahad Arslan]

Signed-off-by: Fahad Arslan <fahad_arslan@mentor.com>
---
 meta/recipes-gnome/libffi/libffi_3.0.13.bb |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/meta/recipes-gnome/libffi/libffi_3.0.13.bb b/meta/recipes-gnome/libffi/libffi_3.0.13.bb
index db0841e..07dd7d1 100644
--- a/meta/recipes-gnome/libffi/libffi_3.0.13.bb
+++ b/meta/recipes-gnome/libffi/libffi_3.0.13.bb
@@ -22,3 +22,8 @@ inherit autotools
 FILES_${PN}-dev += "${libdir}/libffi-${PV}"
 
 BBCLASSEXTEND = "native nativesdk"
+
+do_install_append() {
+    install -d ${D}${includedir}
+    cp -r ${D}${libdir}/libffi-${PV}/include/* ${D}${includedir}/ || true
+}
-- 
1.7.9.5

Re: [PATCH] libffi: Copy headers to more common include path.

[Richard Purdie]

On Thu, 2013-12-12 at 22:47 +0500, Fahad Arslan wrote:
> Signed-off-by: Fahad Arslan <fahad_arslan@mentor.com>
> ---
>  meta/recipes-gnome/libffi/libffi_3.0.13.bb |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/meta/recipes-gnome/libffi/libffi_3.0.13.bb b/meta/recipes-gnome/libffi/libffi_3.0.13.bb
> index db0841e..07dd7d1 100644
> --- a/meta/recipes-gnome/libffi/libffi_3.0.13.bb
> +++ b/meta/recipes-gnome/libffi/libffi_3.0.13.bb
> @@ -22,3 +22,8 @@ inherit autotools
>  FILES_${PN}-dev += "${libdir}/libffi-${PV}"
>  
>  BBCLASSEXTEND = "native nativesdk"
> +
> +do_install_append() {
> +    install -d ${D}${includedir}
> +    cp -r ${D}${libdir}/libffi-${PV}/include/* ${D}${includedir}/ || true
> +}

Why? Shouldn't whatever is using them look for them in the correct
location?

Cheers,

Richard

Re: [PATCH] libffi: Copy headers to more common include path.

[Mark Hatle]

On 12/12/13, 11:57 AM, Richard Purdie wrote:
> On Thu, 2013-12-12 at 22:47 +0500, Fahad Arslan wrote:
>> Signed-off-by: Fahad Arslan <fahad_arslan@mentor.com>
>> ---
>>   meta/recipes-gnome/libffi/libffi_3.0.13.bb |    5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/meta/recipes-gnome/libffi/libffi_3.0.13.bb b/meta/recipes-gnome/libffi/libffi_3.0.13.bb
>> index db0841e..07dd7d1 100644
>> --- a/meta/recipes-gnome/libffi/libffi_3.0.13.bb
>> +++ b/meta/recipes-gnome/libffi/libffi_3.0.13.bb
>> @@ -22,3 +22,8 @@ inherit autotools
>>   FILES_${PN}-dev += "${libdir}/libffi-${PV}"
>>
>>   BBCLASSEXTEND = "native nativesdk"
>> +
>> +do_install_append() {
>> +    install -d ${D}${includedir}
>> +    cp -r ${D}${libdir}/libffi-${PV}/include/* ${D}${includedir}/ || true
>> +}
>
> Why? Shouldn't whatever is using them look for them in the correct
> location?

Correct, I think this is wrong as well.  Most of the time when versions are 
included into the include path, the system may allow for multiple versions to be 
installed and used at the same time.

--Mark

> Cheers,
>
> Richard
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>

Re: [PATCH] libffi: Copy headers to more common include path.

[Arslan, Fahad]

My understanding was that headers should be placed under ${includedir} instead of 
${libdir}. Isn't it the practice or am I missing something?

Thanks,
Fahad
________________________________________
From: Richard Purdie [richard.purdie@linuxfoundation.org]
Sent: Thursday, December 12, 2013 10:57 PM
To: Arslan, Fahad
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] libffi: Copy headers to more common include path.

On Thu, 2013-12-12 at 22:47 +0500, Fahad Arslan wrote:
> Signed-off-by: Fahad Arslan <fahad_arslan@mentor.com>
> ---
>  meta/recipes-gnome/libffi/libffi_3.0.13.bb |    5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/meta/recipes-gnome/libffi/libffi_3.0.13.bb b/meta/recipes-gnome/libffi/libffi_3.0.13.bb
> index db0841e..07dd7d1 100644
> --- a/meta/recipes-gnome/libffi/libffi_3.0.13.bb
> +++ b/meta/recipes-gnome/libffi/libffi_3.0.13.bb
> @@ -22,3 +22,8 @@ inherit autotools
>  FILES_${PN}-dev += "${libdir}/libffi-${PV}"
>
>  BBCLASSEXTEND = "native nativesdk"
> +
> +do_install_append() {
> +    install -d ${D}${includedir}
> +    cp -r ${D}${libdir}/libffi-${PV}/include/* ${D}${includedir}/ || true
> +}

Why? Shouldn't whatever is using them look for them in the correct
location?

Cheers,

Richard

Re: [PATCH] libffi: Copy headers to more common include path.

[Burton, Ross]

On 13 December 2013 11:49, Arslan, Fahad <Fahad_Arslan@mentor.com> wrote:
> My understanding was that headers should be placed under ${includedir} instead of
> ${libdir}. Isn't it the practice or am I missing something?

If the headers are machine-specific then it's conventional to put a
user-visible include $includedir (arch-independent) which then
includes files from $libdir (arch-dependent).

Considering libffi needs to know details of the calling convention on
the machine, this is presumably what's happening.  GLib does the same.

Ross