Skip to contentSkip to conversation

package/ffmpeg: fix arc fenv build failure

PATCH New Bernd Kuhls 2 participants 2 messages Nov 4, 2017

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/ffmpeg: fix arc fenv build failure

Fixes
http://autobuild.buildroot.net/results/aec/aec300d7d4472421398a24202d6871c965bd1be5/

---
 package/ffmpeg/Config.in | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
package/ffmpeg/Config.in +4 -2
index ed87ddd492..c4ce8360a7 100644
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_FFMPEG_ARCH_SUPPORTS
 	bool
-	# fenv.h lacks FE_INVALID, FE_OVERFLOW & FE_UNDERFLOW on nios2
+	# fenv.h lacks FE_INVALID, FE_OVERFLOW & FE_UNDERFLOW on arc
+	# and nios2
 	# No support for ARMv7-M in the ARM assembly logic
 	# Microblaze build affected by gcc PR71124 (infinite loop)
 	# m68k coldfire causes a build failure, because the check for
@@ -9,7 +10,8 @@ config BR2_PACKAGE_FFMPEG_ARCH_SUPPORTS
 	# not. See https://patchwork.ozlabs.org/patch/756664/ and
 	# https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68467 for more
 	# details.
-	default y if !BR2_nios2 && !BR2_ARM_CPU_ARMV7M && !BR2_microblaze && !BR2_m68k_cf
+	default y if !BR2_arc && !BR2_nios2 && !BR2_ARM_CPU_ARMV7M && \
+		!BR2_microblaze && !BR2_m68k_cf
 
 menuconfig BR2_PACKAGE_FFMPEG
 	bool "ffmpeg"
Signature · 1 line
2.11.0
#1 lore raw

PATCH 1/1 package/ffmpeg: fix arc fenv build failure

Hello,

Quoted 22 lines · Bernd Kuhls

Fixes
http://autobuild.buildroot.net/results/aec/aec300d7d4472421398a24202d6871c965bd1be5/

---
 package/ffmpeg/Config.in | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
package/ffmpeg/Config.in +2 -1
index ed87ddd492..c4ce8360a7 100644
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_FFMPEG_ARCH_SUPPORTS
 	bool
-	# fenv.h lacks FE_INVALID, FE_OVERFLOW & FE_UNDERFLOW on nios2
+	# fenv.h lacks FE_INVALID, FE_OVERFLOW & FE_UNDERFLOW on arc
+	# and nios2
 	# No support for ARMv7-M in the ARM assembly logic
 	# Microblaze build affected by gcc PR71124 (infinite loop)
 	# m68k coldfire causes a build failure, because the check for

package/ffmpeg/Config.in config BR2_PACKAGE_FFMPEG_ARCH_SUPPORTS · line 14

Quoted 6 lines
package/ffmpeg/Config.in +2 -1
@@ -9,7 +10,8 @@ config BR2_PACKAGE_FFMPEG_ARCH_SUPPORTS
 	# not. See https://patchwork.ozlabs.org/patch/756664/ and
 	# https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68467 for more
 	# details.
-	default y if !BR2_nios2 && !BR2_ARM_CPU_ARMV7M && !BR2_microblaze && !BR2_m68k_cf
+	default y if !BR2_arc && !BR2_nios2 && !BR2_ARM_CPU_ARMV7M && \
+		!BR2_microblaze && !BR2_m68k_cf

I think only ARC+glibc is affected by this FE_* problem, no? ARC+uClibc does work, no? (That's really a question, it should be verified).

Adding Alexey in Cc. Alexey: that's another example of a package that needs proper <fenv.h> support in glibc.

Thanks,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com