[Gstreamer-bugs] [Bug 99816] New - configure.in shouldn't hardcode Wall

[bugzilla-daemon at widget.gnome.org]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=99816

Changed by brian.cameron at ireland.sun.com.

--- shadow/99816	Thu Nov 28 09:08:52 2002
+++ shadow/99816.tmp.20043	Thu Nov 28 09:08:52 2002
@@ -0,0 +1,41 @@
+Bug#: 99816
+Product: GStreamer
+Version: cvs
+OS: Solaris
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: normal
+Priority: Normal
+Component: gstreamer (core)
+AssignedTo: gstreamer-maint at bugzilla.gnome.org                            
+ReportedBy: brian.cameron at ireland.sun.com               
+QAContact: gstreamer-maint at bugzilla.gnome.org
+TargetMilestone: 0.4.0
+URL: 
+Summary: configure.in shouldn't hardcode Wall
+
+-Wall and -Werror are gcc specific compiler options and should only be
+included if the compiler is gcc.  For example, the glib configure.in
+handles this as follows:
+
+--glib example start--
+
+if test "x$GCC" = "xyes"; then
+  case " $CFLAGS " in
+  *[[\ \    ]]-Wall[[\ \    ]]*) ;;
+  *) CFLAGS="$CFLAGS -Wall" ;;
+  esac
+
+  if test "x$enable_ansi" = "xyes"; then
+    case " $CFLAGS " in
+    *[[\ \  ]]-ansi[[\ \    ]]*) ;;
+    *) CFLAGS="$CFLAGS -ansi" ;;
+    esac
+
+    case " $CFLAGS " in
+    *[[\ \  ]]-pedantic[[\ \	]]*) ;;
+    *) CFLAGS="$CFLAGS -pedantic" ;;
+    esac
+  fi
+fi