diff mbox

freetype: freetype-patch: depends on host-automake

Message ID 1399282934-14517-1-git-send-email-fabio.porcedda@gmail.com
State Superseded
Headers show

Commit Message

Fabio Porcedda May 5, 2014, 9:42 a.m. UTC
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@gmail.com>
---
 package/freetype/freetype.mk | 3 +++
 1 file changed, 3 insertions(+)

Comments

Samuel Martin May 5, 2014, 9:55 a.m. UTC | #1
Hi Fabio, all,

On Mon, May 5, 2014 at 11:42 AM, Fabio Porcedda
<fabio.porcedda@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@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?

Regards
diff mbox

Patch

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