R: use optional dep for valgrind

1 files changed, 6 insertions, 5 deletions

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?