[Buildroot] [PATCH 1/1] libffi: minor corrections in post install hooks

[Buildroot] [PATCH 1/1] libffi: minor corrections in post install hooks

[Nicolas Ménégale]

Signed-off-by: Nicolas M?n?gale <nicolas.menegale@openwide.fr>
---
v1: second commit
In order to be sure that we just deal with the includes related with
the actual package described into libffi.mk it is better to explicitly
specify the version of the libffi we deal with into the post install
hook.
As dev files are deprecated in target rootfs target post install
hook was also deleted.
---
 package/libffi/libffi.mk |   11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/package/libffi/libffi.mk b/package/libffi/libffi.mk
index 0ad5c84..8c84924 100644
--- a/package/libffi/libffi.mk
+++ b/package/libffi/libffi.mk
@@ -14,7 +14,7 @@ LIBFFI_AUTORECONF = YES
 # Move the headers to the usual location, and adjust the .pc file
 # accordingly.
 define LIBFFI_MOVE_STAGING_HEADERS
-	mv $(STAGING_DIR)/usr/lib/libffi-*/include/*.h $(STAGING_DIR)/usr/include/
+	mv $(STAGING_DIR)/usr/lib/libffi-$(LIBFFI_VERSION)/include/*.h $(STAGING_DIR)/usr/include/
 	$(SED) '/^includedir.*/d' -e '/^Cflags:.*/d' \
 		$(STAGING_DIR)/usr/lib/pkgconfig/libffi.pc
 	rm -rf $(TARGET_DIR)/usr/lib/libffi-*
@@ -22,15 +22,6 @@ endef
 
 LIBFFI_POST_INSTALL_STAGING_HOOKS += LIBFFI_MOVE_STAGING_HEADERS
 
-# Similar for target headers
-define LIBFFI_MOVE_TARGET_HEADERS
-	install -d $(TARGET_DIR)/usr/include/
-	mv $(TARGET_DIR)/usr/lib/libffi-*/include/*.h $(TARGET_DIR)/usr/include/
-	rm -rf $(TARGET_DIR)/usr/lib/libffi-*
-endef
-
-LIBFFI_POST_INSTALL_TARGET_HOOKS += LIBFFI_MOVE_TARGET_HEADERS
-
 HOST_LIBFFI_AUTORECONF = YES
 
 $(eval $(autotools-package))
-- 
1.7.9.5

[Buildroot] [PATCH 1/1] libffi: minor corrections in post install hooks

[Thomas Petazzoni]

Dear Nicolas M?n?gale,

This looks good, but:

 * Your patch subject should have mentioned it's the second version.
   See --subject-prefix when generating/sending your patches.

 * The commit log could be a little bit more verbose on what the commit
   is doing. Basically, all the details you wrote after the "---"
   should in this case be part of the commit log itself. What is after
   the "---" marker are changelog informations, i.e things that changed
   between revisions of the patches. The real description of what the
   patch is doing should be part of the commit log.

Thanks,

Thomas

On Tue, 27 Aug 2013 14:26:49 +0200 (CEST), Nicolas M?n?gale wrote:
> Signed-off-by: Nicolas M?n?gale <nicolas.menegale@openwide.fr>
> ---
> v1: second commit
> In order to be sure that we just deal with the includes related with
> the actual package described into libffi.mk it is better to explicitly
> specify the version of the libffi we deal with into the post install
> hook.
> As dev files are deprecated in target rootfs target post install
> hook was also deleted.
> ---
>  package/libffi/libffi.mk |   11 +----------
>  1 file changed, 1 insertion(+), 10 deletions(-)
> 
> diff --git a/package/libffi/libffi.mk b/package/libffi/libffi.mk
> index 0ad5c84..8c84924 100644
> --- a/package/libffi/libffi.mk
> +++ b/package/libffi/libffi.mk
> @@ -14,7 +14,7 @@ LIBFFI_AUTORECONF = YES
>  # Move the headers to the usual location, and adjust the .pc file
>  # accordingly.
>  define LIBFFI_MOVE_STAGING_HEADERS
> -	mv $(STAGING_DIR)/usr/lib/libffi-*/include/*.h $(STAGING_DIR)/usr/include/
> +	mv $(STAGING_DIR)/usr/lib/libffi-$(LIBFFI_VERSION)/include/*.h $(STAGING_DIR)/usr/include/
>  	$(SED) '/^includedir.*/d' -e '/^Cflags:.*/d' \
>  		$(STAGING_DIR)/usr/lib/pkgconfig/libffi.pc
>  	rm -rf $(TARGET_DIR)/usr/lib/libffi-*
> @@ -22,15 +22,6 @@ endef
>  
>  LIBFFI_POST_INSTALL_STAGING_HOOKS += LIBFFI_MOVE_STAGING_HEADERS
>  
> -# Similar for target headers
> -define LIBFFI_MOVE_TARGET_HEADERS
> -	install -d $(TARGET_DIR)/usr/include/
> -	mv $(TARGET_DIR)/usr/lib/libffi-*/include/*.h $(TARGET_DIR)/usr/include/
> -	rm -rf $(TARGET_DIR)/usr/lib/libffi-*
> -endef
> -
> -LIBFFI_POST_INSTALL_TARGET_HOOKS += LIBFFI_MOVE_TARGET_HEADERS
> -
>  HOST_LIBFFI_AUTORECONF = YES
>  
>  $(eval $(autotools-package))



-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH 1/1] libffi: minor corrections in post install hooks

[Nicolas Ménégale]

Signed-off-by: Nicolas M?n?gale <nicolas.menegale@openwide.fr>
---
v0: initial commit
In order to be sure that we just deal with the includes related with
the actual package described into libffi.mk it is better to explicitly
specify the version of the libffi we deal with into the post install
hooks.
---
 package/libffi/libffi.mk |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/libffi/libffi.mk b/package/libffi/libffi.mk
index 0ad5c84..96e7940 100644
--- a/package/libffi/libffi.mk
+++ b/package/libffi/libffi.mk
@@ -14,7 +14,7 @@ LIBFFI_AUTORECONF = YES
 # Move the headers to the usual location, and adjust the .pc file
 # accordingly.
 define LIBFFI_MOVE_STAGING_HEADERS
-	mv $(STAGING_DIR)/usr/lib/libffi-*/include/*.h $(STAGING_DIR)/usr/include/
+	mv $(STAGING_DIR)/usr/lib/libffi-$(LIBFFI_VERSION)/include/*.h $(STAGING_DIR)/usr/include/
 	$(SED) '/^includedir.*/d' -e '/^Cflags:.*/d' \
 		$(STAGING_DIR)/usr/lib/pkgconfig/libffi.pc
 	rm -rf $(TARGET_DIR)/usr/lib/libffi-*
@@ -25,7 +25,7 @@ LIBFFI_POST_INSTALL_STAGING_HOOKS += LIBFFI_MOVE_STAGING_HEADERS
 # Similar for target headers
 define LIBFFI_MOVE_TARGET_HEADERS
 	install -d $(TARGET_DIR)/usr/include/
-	mv $(TARGET_DIR)/usr/lib/libffi-*/include/*.h $(TARGET_DIR)/usr/include/
+	mv $(TARGET_DIR)/usr/lib/libffi-$(LIBFFI_VERSION)/include/*.h $(TARGET_DIR)/usr/include/
 	rm -rf $(TARGET_DIR)/usr/lib/libffi-*
 endef
 
-- 
1.7.9.5

[Buildroot] [PATCH 1/1] libffi: minor corrections in post install hooks

[Samuel Martin]

Hi Nicolas,


2013/8/27 Nicolas M?n?gale <nicolas.menegale@openwide.fr>

> Signed-off-by: Nicolas M?n?gale <nicolas.menegale@openwide.fr>
> ---
> v0: initial commit
> In order to be sure that we just deal with the includes related with
> the actual package described into libffi.mk it is better to explicitly
> specify the version of the libffi we deal with into the post install
> hooks.
> ---
>  package/libffi/libffi.mk |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/libffi/libffi.mk b/package/libffi/libffi.mk
> index 0ad5c84..96e7940 100644
> --- a/package/libffi/libffi.mk
> +++ b/package/libffi/libffi.mk
> @@ -14,7 +14,7 @@ LIBFFI_AUTORECONF = YES
>  # Move the headers to the usual location, and adjust the .pc file
>  # accordingly.
>  define LIBFFI_MOVE_STAGING_HEADERS
> -       mv $(STAGING_DIR)/usr/lib/libffi-*/include/*.h
> $(STAGING_DIR)/usr/include/
> +       mv $(STAGING_DIR)/usr/lib/libffi-$(LIBFFI_VERSION)/include/*.h
> $(STAGING_DIR)/usr/include/
>         $(SED) '/^includedir.*/d' -e '/^Cflags:.*/d' \
>                 $(STAGING_DIR)/usr/lib/pkgconfig/libffi.pc
>         rm -rf $(TARGET_DIR)/usr/lib/libffi-*
> @@ -25,7 +25,7 @@ LIBFFI_POST_INSTALL_STAGING_HOOKS +=
> LIBFFI_MOVE_STAGING_HEADERS
>  # Similar for target headers
>  define LIBFFI_MOVE_TARGET_HEADERS
>         install -d $(TARGET_DIR)/usr/include/
> -       mv $(TARGET_DIR)/usr/lib/libffi-*/include/*.h
> $(TARGET_DIR)/usr/include/
> +       mv $(TARGET_DIR)/usr/lib/libffi-$(LIBFFI_VERSION)/include/*.h
> $(TARGET_DIR)/usr/include/
>         rm -rf $(TARGET_DIR)/usr/lib/libffi-*
>  endef
>
Since we deprecated dev. files installation in the target rootfs and we no
longer
support building gcc for the target, I think you can simply remove the
LIBFFI_MOVE_TARGET_HEADERS hook.

Regards,

-- 
Samuel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20130827/618644c3/attachment.html>