[Buildroot] [PATCH] x264: fix build on OpenRISC

[Buildroot] [PATCH] x264: fix build on OpenRISC

[Thomas Petazzoni]

Even though x264 doesn't use an autoconf generated configure script
(and hence doesn't use the autotools-package infrastructure), it uses
config.guess and config.sub. The ones bundled in x264 are too old to
know about the OpenRISC architecture, so this commit leverages the
UPDATE_CONFIG_HOOK from the autotools-package infrastructure to update
config.sub and config.guess.

Fixes:

  http://autobuild.buildroot.net/results/b458c2dff0ecde61248f058092c6eeaa75039cfb

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

diff --git a/package/x264/x264.mk b/package/x264/x264.mk
index 80437c4..2c8040b 100644
--- a/package/x264/x264.mk
+++ b/package/x264/x264.mk
@@ -36,6 +36,11 @@ ifeq ($(BR2_TOOLCHAIN_HAS_THREADS),)
 X264_CONF_OPTS += --disable-thread
 endif
 
+# Even though the configure script is not generated by autoconf, x264
+# uses config.sub/config.guess, so we want up-to-date versions of
+# them.
+X264_POST_PATCH_HOOKS += UPDATE_CONFIG_HOOK
+
 # the configure script is not generated by autoconf
 define X264_CONFIGURE_CMDS
 	(cd $(@D); $(TARGET_CONFIGURE_OPTS) $(X264_CONF_ENV) ./configure \
-- 
2.7.4

[Buildroot] [PATCH] x264: fix build on OpenRISC

[Thomas Petazzoni]

Hello,

On Sat,  4 Mar 2017 14:45:50 +0100, Thomas Petazzoni wrote:
> Even though x264 doesn't use an autoconf generated configure script
> (and hence doesn't use the autotools-package infrastructure), it uses
> config.guess and config.sub. The ones bundled in x264 are too old to
> know about the OpenRISC architecture, so this commit leverages the
> UPDATE_CONFIG_HOOK from the autotools-package infrastructure to update
> config.sub and config.guess.
> 
> Fixes:
> 
>   http://autobuild.buildroot.net/results/b458c2dff0ecde61248f058092c6eeaa75039cfb
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  package/x264/x264.mk | 5 +++++
>  1 file changed, 5 insertions(+)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com