From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] ffmpeg: explicitly pass --ranlib
Date: Thu, 16 Apr 2015 23:12:10 +0200 [thread overview]
Message-ID: <20150416211210.GF4405@free.fr> (raw)
In-Reply-To: <1429214355-29429-1-git-send-email-arnout@mind.be>
Arnout, All,
On 2015-04-16 21:59 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
> It seems that ffmpeg's custom configure doesn't observe the RANLIB
> environment variable, and incorrectly uses the host's ranlib to
> check if the -D option is supported. Blackfin's ranlib doesn't, most
> others do.
>
> I also checked if there are any more mistakes like that in ffmpeg's
> configure script, but it looked OK.
Your wording "ffmpeg's configure" prompted me to look at ffmpeg, and
so here's my thoughts about it...
Given that ffmpeg is not using autoconf, automake and co., it should not
be usign the autotools-package infra, but should be switched over to
using the generic-package infra instead.
OK, not your fault! ;-)
Regards,
Yann E. MORIN.
> Fixes:
> http://autobuild.buildroot.net/results/d13488a43f6e034623104a36c07114db99325b8d
>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> Cc: Bernd Kuhs <bernd.kuhls@t-online.de>
> ---
> package/ffmpeg/ffmpeg.mk | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/package/ffmpeg/ffmpeg.mk b/package/ffmpeg/ffmpeg.mk
> index 787dbb1..59c49ee 100644
> --- a/package/ffmpeg/ffmpeg.mk
> +++ b/package/ffmpeg/ffmpeg.mk
> @@ -18,6 +18,7 @@ endif
>
> FFMPEG_CONF_OPTS = \
> --prefix=/usr \
> + --ranlib=$(TARGET_RANLIB) \
> --enable-avfilter \
> --disable-version3 \
> --enable-logging \
> --
> 2.1.4
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
prev parent reply other threads:[~2015-04-16 21:12 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-16 19:59 [Buildroot] [PATCH] ffmpeg: explicitly pass --ranlib Arnout Vandecappelle
2015-04-16 20:17 ` Bernd Kuhls
2015-04-16 22:02 ` Arnout Vandecappelle
2015-04-16 21:12 ` Yann E. MORIN [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150416211210.GF4405@free.fr \
--to=yann.morin.1998@free.fr \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox