[FFmpeg-devel] [PATCH] make configure error out if endianness detection fails

Reimar Döffinger Reimar.Doeffinger
Wed Sep 30 17:20:34 CEST 2009


Hello,
currently if e.g. the compiler manages to completely optimize away the
bigendian test, we will compile for little endian.
This patch makes configure error out instead.
I'd also like to suggest to remove "-A n", the test IMO _should_ work
without and it would move configure closer to working on OpenBSD (the
only other issue I know is that OpenBSD nm does not know -P).
Index: configure
===================================================================
--- configure   (revision 20097)
+++ configure   (working copy)
@@ -2090,6 +2090,7 @@
 unsigned int endian = 'B' << 24 | 'I' << 16 | 'G' << 8 | 'E';
 EOF
 od -A n -t x1 $TMPO | grep -q '42 *49 *47 *45' && enable bigendian
+od -A n -t x1 $TMPO | grep -q '45 *47 *49 *42' || enabled bigendian || die "unknown endianness"
 
 if enabled arm; then
 




More information about the ffmpeg-devel mailing list