diff mbox

[4/6] gutenprint: ensure target and host variant have the same patch hooks

Message ID 1369695664-5670-5-git-send-email-thomas.petazzoni@free-electrons.com
State Accepted
Headers show

Commit Message

Thomas Petazzoni May 27, 2013, 11:01 p.m. UTC
In preparation for the out-of-tree support, it is important that all
packages have the same patch hooks for the host variant and the target
variant (because the source tree will be shared between host and
target variants when doing out-of-tree build).

In the case of gutenprint, the patch hook executed for the target just
creates one directory which is needed for autoreconf to work, and the
host variant is not autoreconf'ed. So there is no harm in also
applying this patch hook to the host variant.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/gutenprint/gutenprint.mk |    1 +
 1 file changed, 1 insertion(+)

Comments

Arnout Vandecappelle May 28, 2013, 8:11 p.m. UTC | #1
On 28/05/13 01:01, Thomas Petazzoni wrote:
> In the case of gutenprint, the patch hook executed for the target just
> creates one directory which is needed for autoreconf to work, and the
> host variant is not autoreconf'ed. So there is no harm in also
> applying this patch hook to the host variant.

  Since autoreconf is going to be done on the source tree, you should 
probably autoreconf the host variant as well.

  Regards,
  Arnout
Peter Korsgaard May 30, 2013, 7:54 p.m. UTC | #2
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> In preparation for the out-of-tree support, it is important that all
 Thomas> packages have the same patch hooks for the host variant and the target
 Thomas> variant (because the source tree will be shared between host and
 Thomas> target variants when doing out-of-tree build).

 Thomas> In the case of gutenprint, the patch hook executed for the target just
 Thomas> creates one directory which is needed for autoreconf to work, and the
 Thomas> host variant is not autoreconf'ed. So there is no harm in also
 Thomas> applying this patch hook to the host variant.

Committed to next with HOST_GUTENPRINT_AUTORECONF = YES as pointed out
by Arnout, thanks.
diff mbox

Patch

diff --git a/package/gutenprint/gutenprint.mk b/package/gutenprint/gutenprint.mk
index 834d967..2431a54 100644
--- a/package/gutenprint/gutenprint.mk
+++ b/package/gutenprint/gutenprint.mk
@@ -69,6 +69,7 @@  define GUTENPRINT_CREATE_M4_DIR
 	mkdir -p $(@D)/m4local
 endef
 GUTENPRINT_POST_PATCH_HOOKS += GUTENPRINT_CREATE_M4_DIR
+HOST_GUTENPRINT_POST_PATCH_HOOKS += GUTENPRINT_CREATE_M4_DIR
 
 define HOST_GUTENPRINT_POST_BUILD_INSTAL_TMP_HEADER
 	cp $(@D)/src/xml/xmli18n-tmp.h $(HOST_DIR)/usr/include