From 6671d6275de7bcb9733b20aca8ea930abb5013d9 Mon Sep 17 00:00:00 2001
From: Adam Vandenberg
Date: Tue, 9 Feb 2010 09:21:25 -0800
Subject: remove_make_var! should not leave an empty assignment

---
 Library/Homebrew/utils.rb | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

(limited to 'Library/Homebrew/utils.rb')

diff --git a/Library/Homebrew/utils.rb b/Library/Homebrew/utils.rb
index ae6ca50b9..3f89e187a 100644
--- a/Library/Homebrew/utils.rb
+++ b/Library/Homebrew/utils.rb
@@ -177,16 +177,20 @@ def archs_for_command cmd
     end
 end
 
+# String extensions added by inreplace below.
 module HomebrewInreplaceExtension
   # Looks for Makefile style variable defintions and replaces the
   # value with "new_value", or removes the definition entirely.
-  # See inreplace in utils.rb
-  def change_make_var! flag, new_value=nil
-    new_value = "#{flag}=#{new_value}" unless new_value == nil
+  def change_make_var! flag, new_value
+    new_value = "#{flag}=#{new_value}"
     gsub! Regexp.new("^#{flag}\\s*=\\s*(.*)$"), new_value
   end
+  # Removes variable assignments completely.
   def remove_make_var! flags
-    flags.each { |flag| change_make_var! flag, "" }
+    flags.each do |flag|
+      # Also remove trailing \n, if present.
+      gsub! Regexp.new("^#{flag}\\s*=(.*)$\n?"), ""
+    end
   end
 end
 
-- 
cgit v1.2.3