From 3dbc905676a53acc3a5ad88af89cf4ffbc502a33 Mon Sep 17 00:00:00 2001
From: Jack Nagel
Date: Tue, 25 Jun 2013 21:17:59 -0500
Subject: R: use optional dep for valgrind

---
 Library/Formula/r.rb | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

(limited to 'Library')

diff --git a/Library/Formula/r.rb b/Library/Formula/r.rb
index 3c9aa3867..1158109d5 100644
--- a/Library/Formula/r.rb
+++ b/Library/Formula/r.rb
@@ -22,19 +22,20 @@ class R < Formula
   depends_on 'libtiff'
   depends_on 'jpeg'
   depends_on :x11
-
-  depends_on 'valgrind' if build.include? 'with-valgrind'
+  depends_on 'valgrind' => :optional
 
   def install
-    ENV.Og if build.include? 'with-valgrind'
-
     args = [
       "--prefix=#{prefix}",
       "--with-aqua",
       "--enable-R-framework",
       "--with-lapack"
     ]
-    args << '--with-valgrind-instrumentation=2' if build.include? 'with-valgrind'
+
+    if build.with? 'valgrind'
+      args << '--with-valgrind-instrumentation=2'
+      ENV.Og
+    end
 
     # Pull down recommended packages if building from HEAD.
     system './tools/rsync-recommended' if build.head?
-- 
cgit v1.2.3