Updated ffmpeg formula

* now supports, x264, faac, faad, lame optional dependencies.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>

* Also silenced a warning on x86_64.

1 files changed, 26 insertions, 8 deletions

diff --git a/Library/Formula/ffmpeg.rb b/Library/Formula/ffmpeg.rb
index a16802433..24954e3e5 100644
--- a/Library/Formula/ffmpeg.rb
+++ b/Library/Formula/ffmpeg.rb
@@ -4,17 +4,35 @@ class Ffmpeg <Formula
   head 'svn://svn.ffmpeg.org/ffmpeg/trunk', :revision => 22585
   homepage 'http://ffmpeg.org/'
 
-  depends_on 'x264'
+  depends_on 'x264' => :optional
+  depends_on 'faac' => :optional
+  depends_on 'faad2' => :optional
+  depends_on 'lame' => :optional
 
   def install
-    configure_flags = [ "--prefix=#{prefix}",
-                          "--disable-debug",
-                          "--enable-shared",
-                          "--enable-pthreads",
-                          "--enable-nonfree",
-                          "--enable-gpl",
-                          "--enable-libx264"]
+    configure_flags = [
+      "--prefix=#{prefix}",
+      "--disable-debug",
+      "--enable-shared",
+      "--enable-pthreads",
+      "--enable-nonfree",
+      "--enable-gpl"
+    ]
+
+    configure_flags << "--enable-libx264" if Formula.factory('x264').installed?
+    configure_flags << "--enable-libfaac" if Formula.factory('faac').installed?
+    configure_flags << "--enable-libfaad" if Formula.factory('faad2').installed?
+    configure_flags << "--enable-libmp3lame" if Formula.factory('lame').installed?
+
     system "./configure", *configure_flags
+
+    inreplace 'config.mak' do |s|
+      if MACOS_VERSION >= 10.6 and Hardware.is_64_bit?
+        shflags = s.get_make_var 'SHFLAGS'
+        s.change_make_var! 'SHFLAGS', shflags.gsub!(' -Wl,-read_only_relocs,suppress', '')
+      end
+    end
+
     system "make install"
   end
 end