[MPlayer-cvslog] r35172 - trunk/configure

reimar subversion at mplayerhq.hu
Thu Sep 13 00:01:36 CEST 2012


Author: reimar
Date: Thu Sep 13 00:01:36 2012
New Revision: 35172

Log:
Pass input to "strings" via stdin.

That disables any "clever" parsing that will fail if strings
was not updated to support that particular binary format.
This can easily happen when cross-compiling, but obviously
strings on OSX 10.6 was never update to handle the latest
(64 bit) binary format either and would previously just fail
(the obvious way of falling back to the generic algorithm in
that case would have been too user-friendly I guess...)

Modified:
   trunk/configure

Modified: trunk/configure
==============================================================================
--- trunk/configure	Wed Sep 12 23:03:45 2012	(r35171)
+++ trunk/configure	Thu Sep 13 00:01:36 2012	(r35172)
@@ -2540,7 +2540,9 @@ short ascii_name[] = {
 int main(void) { return (long)ascii_name; }
 EOF
   if cc_check ; then
-    if strings $TMPEXE | grep -q -l MPlayerBigEndian ; then
+    # stdin is used to make "strings" not try something clever like
+    # parse executable section headers (and possibly fail in the process)
+    if strings < $TMPEXE | grep -q -l MPlayerBigEndian ; then
       _big_endian=yes
     else
       _big_endian=no


More information about the MPlayer-cvslog mailing list