[Buildroot] [PATCH] freetype: freetype-patch: depends on host-automake
Fabio Porcedda
fabio.porcedda at gmail.com
Mon May 5 10:01:08 UTC 2014
On Mon, May 5, 2014 at 11:55 AM, Samuel Martin <s.martin49 at gmail.com> wrote:
> Hi Fabio, all,
>
> On Mon, May 5, 2014 at 11:42 AM, Fabio Porcedda
> <fabio.porcedda at gmail.com> wrote:
>> The "make freetype-patch" fails because does not found autogen.sh,
>> because autogen.sh available only for the configure stage.
>> To fix this add host-automake as a prerequisite to freetype-patch too.
>>
>> This also fix the support to top-level parallel make.
>>
>> Signed-off-by: Fabio Porcedda <fabio.porcedda at gmail.com>
>> ---
>> package/freetype/freetype.mk | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/package/freetype/freetype.mk b/package/freetype/freetype.mk
>> index c7c9a06..33a6317 100644
>> --- a/package/freetype/freetype.mk
>> +++ b/package/freetype/freetype.mk
>> @@ -83,3 +83,6 @@ HOST_FREETYPE_POST_INSTALL_HOOKS += HOST_FREETYPE_FIX_FREETYPE_INCLUDE
>>
>> $(eval $(autotools-package))
>> $(eval $(host-autotools-package))
>> +
>> +# freetype-patch uses autogen.sh so add it as a prerequisite
>> +$(FREETYPE_TARGET_PATCH): host-automake
>
> It's an unusual way of doing this!
> How about moving the *FREETYPE_RUN_AUTOGEN hook in
> *FREETYPE_PRE_CONFIGURE_HOOKS?
Unfortunately that's alone does not work because LIBTOOL_PATCH_HOOK
it's going to fail, the only solution i've found is to move
LIBTOOL_PATCH_HOOK to PRE_CONFIGURE too, but i'm not sure is it fine
to do that, what do you think about that?
patch:
diff --git a/package/freetype/freetype.mk b/package/freetype/freetype.mk
index c7c9a06..8aac8a2 100644
--- a/package/freetype/freetype.mk
+++ b/package/freetype/freetype.mk
@@ -25,8 +25,8 @@ HOST_FREETYPE_CONF_OPT = --without-zlib
--without-bzip2 --without-png
define FREETYPE_RUN_AUTOGEN
cd $(@D) && PATH=$(BR_PATH) ./autogen.sh
endef
-FREETYPE_POST_PATCH_HOOKS += FREETYPE_RUN_AUTOGEN
-HOST_FREETYPE_POST_PATCH_HOOKS += FREETYPE_RUN_AUTOGEN
+FREETYPE_PRE_CONFIGURE_HOOKS += FREETYPE_RUN_AUTOGEN
+HOST_FREETYPE_PRE_CONFIGURE_HOOKS += FREETYPE_RUN_AUTOGEN
FREETYPE_DEPENDENCIES += host-automake host-autoconf host-libtool
HOST_FREETYPE_DEPENDENCIES += host-automake host-autoconf host-libtool
diff --git a/package/pkg-autotools.mk b/package/pkg-autotools.mk
index a646612..38b8c4c 100644
--- a/package/pkg-autotools.mk
+++ b/package/pkg-autotools.mk
@@ -193,7 +193,7 @@ endef
# default values are not evaluated yet, so don't rely on this defaulting to YES
ifneq ($$($(2)_LIBTOOL_PATCH),NO)
-$(2)_POST_PATCH_HOOKS += LIBTOOL_PATCH_HOOK
+$(2)_PRE_CONFIGURE_HOOKS += LIBTOOL_PATCH_HOOK
endif
#
Regards
--
Fabio Porcedda
More information about the buildroot
mailing list