ffmpeg: disable for mips64r2 and mips64r6
PATCH 1/1 ffmpeg: disable for mips64r2 and mips64r6
MIPS architecture detection is not accurate and is always detected as mips64 even if we are using mips64r2 or mips64r6. Due to that, ffmpeg's build system will pass the -mips64 flag which will conflict with the -march option that our toolchain wrapper uses, and it will fail to build showing errors like this one:
error: '-mips64' conflicts with the other architecture options, which specify a mips64r2 processor
This problem has been already fixed upstream, but we would need to backport 17 patches plus some changes in the ffmpeg.mk file. This is too much, so better to just disable ffmpeg for mips64r2 and mips64r6 for the upcoming Buildroot release.
This commit can be reverted in the next ffmpeg's version bump.
Fixes:
http://autobuild.buildroot.net/results/7fd/7fd8187c0110cdcac622e667f4a81d2db84f11ef/
--- package/ffmpeg/Config.in | 4 ++++ 1 file changed, 4 insertions(+)
package/ffmpeg/Config.in +4 -0
@@ -2,6 +2,10 @@ menuconfig BR2_PACKAGE_FFMPEG bool "ffmpeg" # fenv.h lacks FE_INVALID, FE_OVERFLOW & FE_UNDERFLOW on nios2 depends on !BR2_nios2 + # MIPS architecture detection is not accurate for 64-bit + # variants. This has been already fixed upstream, so please + # remove it in the next version bump. + depends on !BR2_mips_64r2 && !BR2_mips_64r6 help FFmpeg is a complete, cross-platform solution to record, convert and stream audio and video.
Signature · 1 line
2.7.3
PATCH 1/1 ffmpeg: disable for mips64r2 and mips64r6
Hi,
Am Wed, 25 May 2016 17:14:12 +0100 schrieb Vicente Olivert Riera:
Quoted 1 line
This commit can be reverted in the next ffmpeg's version bump.
[...]
package/ffmpeg/Config.in menuconfig BR2_PACKAGE_FFMPEG · line 8
Quoted 4 lines
package/ffmpeg/Config.in +4
-0+ # MIPS architecture detection is not accurate for 64-bit + # variants. This has been already fixed upstream, so please + # remove it in the next version bump. + depends on !BR2_mips_64r2 && !BR2_mips_64r6
you need to propagate this exception as reverse dependency to the following packages:
- gst1-libav
- kodi
- minidlna
- mpd
- mpv
- opencv
- opencv3
- squeezelite
- tovid
Imho this is too much work, we should live with the build error and bump ffmpeg to 3.0.2 after the next buildroot release to fix the problem. On github you can test my patch series already:
https://github.com/bkuhls/buildroot/tree/ffmpeg3
The main problem is that vlc 2.2.x is incompatible with ffmpeg3, only vlc master branch works, this branch will be released as vlc 3.0 sometime in the future. We would carry a pre-released version of vlc in buildroot, which is something we want to avoid if possible.
Before the ffmpeg 3 vs. vlc problem arose I thought we could remove the vlc warning in its Config.in:
https://git.busybox.net/buildroot/tree/package/vlc/Config.in#n2
When bumping to a pre-release version we should keep it ;)
Regards, Bernd
PATCH 1/1 ffmpeg: disable for mips64r2 and mips64r6
Hello Bernd,
Quoted 24 lines · Bernd Kuhls
Hi,
Am Wed, 25 May 2016 17:14:12 +0100 schrieb Vicente Olivert Riera:
Quoted 1 line
This commit can be reverted in the next ffmpeg's version bump.
[...]
package/ffmpeg/Config.in menuconfig BR2_PACKAGE_FFMPEG · line 8
Quoted 4 lines
package/ffmpeg/Config.in +4
-0+ # MIPS architecture detection is not accurate for 64-bit + # variants. This has been already fixed upstream, so please + # remove it in the next version bump. + depends on !BR2_mips_64r2 && !BR2_mips_64r6you need to propagate this exception as reverse dependency to the following packages:
- gst1-libav
- kodi
- minidlna
- mpd
- mpv
- opencv
- opencv3
- squeezelite
- tovid
Quoted 5 lines
Imho this is too much work, we should live with the build error and bump ffmpeg to 3.0.2 after the next buildroot release to fix the problem. On github you can test my patch series already:
I can mark this patch as rejected and live with the build error by now.
Quoted 10 lines
The main problem is that vlc 2.2.x is incompatible with ffmpeg3, only vlc master branch works, this branch will be released as vlc 3.0 sometime in the future. We would carry a pre-released version of vlc in buildroot, which is something we want to avoid if possible.
Before the ffmpeg 3 vs. vlc problem arose I thought we could remove the vlc warning in its Config.in:
https://git.busybox.net/buildroot/tree/package/vlc/Config.in#n2When bumping to a pre-release version we should keep it ;)
I was wondering why you marked as superseded your patches to bump ffmpeg.
Ok, then let's wait.
Regards,
Vincent.
Quoted 7 lines
Regards, Bernd
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot
PATCH 1/1 ffmpeg: disable for mips64r2 and mips64r6
In reply to Bernd KuhlsHello,
Quoted 17 lines · Bernd Kuhls
package/ffmpeg/Config.in menuconfig BR2_PACKAGE_FFMPEG · line 8
Quoted 4 lines
package/ffmpeg/Config.in +4
-0+ # MIPS architecture detection is not accurate for 64-bit + # variants. This has been already fixed upstream, so please + # remove it in the next version bump. + depends on !BR2_mips_64r2 && !BR2_mips_64r6you need to propagate this exception as reverse dependency to the following packages:
- gst1-libav
- kodi
- minidlna
- mpd
- mpv
- opencv
- opencv3
- squeezelite
- tovid
Quoted 3 lines
Imho this is too much work, we should live with the build error and bump ffmpeg to 3.0.2 after the next buildroot release to fix the problem. On github you can test my patch series already:
ffmpeg already has an architecture exception for nios2. So we could also replace that with a BR2_PACKAGE_FFMPEG_ARCH_SUPPORTS option, and therefore easily add the exception for mips64r2/r6.
We're down to very few build failures per day, so I would really like to get this issue out of the way, and see what other failures we have. ffmpeg on mips64r2/r6 is now the #1 failure.
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
PATCH 1/1 ffmpeg: disable for mips64r2 and mips64r6
Hello Thomas,
Quoted 21 lines · Thomas Petazzoni
Hello,
Quoted 17 lines · Bernd Kuhls
package/ffmpeg/Config.in menuconfig BR2_PACKAGE_FFMPEG · line 8
Quoted 4 lines
package/ffmpeg/Config.in +4
-0+ # MIPS architecture detection is not accurate for 64-bit + # variants. This has been already fixed upstream, so please + # remove it in the next version bump. + depends on !BR2_mips_64r2 && !BR2_mips_64r6you need to propagate this exception as reverse dependency to the following packages:
- gst1-libav
- kodi
- minidlna
- mpd
- mpv
- opencv
- opencv3
- squeezelite
- tovid
Quoted 7 lines
Quoted 3 lines
Imho this is too much work, we should live with the build error and bump ffmpeg to 3.0.2 after the next buildroot release to fix the problem. On github you can test my patch series already:
ffmpeg already has an architecture exception for nios2. So we could also replace that with a
BR2_PACKAGE_FFMPEG_ARCH_SUPPORTSoption, and therefore easily add the exception for mips64r2/r6.
Done and marked this patch as superseded.
Regards,
Vincent.
Quoted 6 lines
We're down to very few build failures per day, so I would really like to get this issue out of the way, and see what other failures we have. ffmpeg on mips64r2/r6 is now the #1 failure.
Thomas