[firefox] Fix up optimize flags

[Christopher Aillon]

commit ed92d8077dcfb9ec455f441f8277ef2e634f6989
Author: Christopher Aillon <caillon at redhat.com>
Date:   Tue Jan 25 11:18:34 2011 -0800

    Fix up optimize flags
    
    Don't pass the opt flags twice to moz, as that will just cause them
    to be duplicated in the compile lines.
    
    Remove commented out optimization flags in the mozconfig
    
    Also add a comment about why we disable cpp exceptions

 firefox-mozconfig |    4 +---
 firefox.spec      |    6 +++++-
 2 files changed, 6 insertions(+), 4 deletions(-)
---
diff --git a/firefox-mozconfig b/firefox-mozconfig
index 3ff8c29..09cedb5 100755
--- a/firefox-mozconfig
+++ b/firefox-mozconfig
@@ -10,9 +10,7 @@ ac_add_options --with-system-zlib
 ac_add_options --with-pthreads
 ac_add_options --disable-tests
 ac_add_options --disable-debug
-ac_add_options --enable-optimize="$MOZ_OPT_FLAGS"
-#ac_add_options --enable-debug
-#ac_add_options --disable-optimize
+ac_add_options --enable-optimize
 ac_add_options --disable-installer
 ac_add_options --enable-xinerama
 ac_add_options --enable-default-toolkit=cairo-gtk2
diff --git a/firefox.spec b/firefox.spec
index db23a55..be56c62 100644
--- a/firefox.spec
+++ b/firefox.spec
@@ -145,7 +145,11 @@ cd %{tarballdir}
 
 # Mozilla builds with -Wall with exception of a few warnings which show up
 # everywhere in the code; so, don't override that.
-export MOZ_OPT_FLAGS=$(echo $RPM_OPT_FLAGS | %{__sed} -e 's/-Wall//' | %{__sed} -e 's/-fexceptions/-fno-exceptions/g')
+#
+# Disable C++ exceptions since Mozilla code is not exception-safe
+#
+MOZ_OPT_FLAGS=$(echo $RPM_OPT_FLAGS | \
+                     %{__sed} -e 's/-Wall//' -e 's/-fexceptions/-fno-exceptions/g')
 export CFLAGS=$MOZ_OPT_FLAGS
 export CXXFLAGS=$MOZ_OPT_FLAGS