[MPlayer-dev-eng] [PATCH] move configure CFLAGS warning at end

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sun Jul 10 11:26:30 CEST 2005


Hi,
since at least one user has managed not to see it, I suggest moving the
CFLAGS-set-warning to the very end, like the attached patch does.
Okay to apply?

Greetings,
Reimar Döffinger
-------------- next part --------------
Index: configure
===================================================================
RCS file: /cvsroot/mplayer/main/configure,v
retrieving revision 1.1026
diff -u -r1.1026 configure
--- configure	26 Jun 2005 17:32:59 -0000	1.1026
+++ configure	30 Jun 2005 19:40:39 -0000
@@ -6655,16 +6678,7 @@
     _stripbinaries=no
   fi
 else
-  cat <<EOF
-
-MPlayer compilation will use the CFLAGS set by you, but:
-
-    *** ***  DO NOT REPORT BUGS IF IT DOES NOT COMPILE/WORK!  *** ***
-
-It is strongly recommended to let MPlayer choose the correct CFLAGS!
-To do so, execute 'CFLAGS= ./configure <options>'
-
-EOF
+  _warn_CFLAGS=yes
 fi
 
 echocheck "automatic gdb attach"
@@ -7837,5 +7863,18 @@
 EOF
 fi
 
+if test "$_warn_CFLAGS" = yes; then
+  cat <<EOF
+
+MPlayer compilation will use the CFLAGS set by you, but:
+
+    *** ***  DO NOT REPORT BUGS IF IT DOES NOT COMPILE/WORK!  *** ***
+
+It is strongly recommended to let MPlayer choose the correct CFLAGS!
+To do so, execute 'CFLAGS= ./configure <options>'
+
+EOF
+fi
+
 # Last move:
 rm -f "$TMPO" "$TMPC" "$TMPS" "$TMPCPP"


More information about the MPlayer-dev-eng mailing list