Use ENV[EDITOR] if possible

Defaulting to EDITOR, then checking for the mate command, and then using
vim as a last resort.

Signed Off By: Max Howell <max@methylblue.com>

Plain brew edit still uses Textmate though because a client that
supported a project concept is required for that particular feature.
Patches for that welcome.

1 files changed, 12 insertions, 0 deletions

diff --git a/Library/Homebrew/utils.rb b/Library/Homebrew/utils.rb
index 8bd2537e5..0dd26d49b 100644
--- a/Library/Homebrew/utils.rb
+++ b/Library/Homebrew/utils.rb
@@ -80,3 +80,15 @@ def puts_columns items, cols = 4
   width=`stty size`.chomp.split(" ").last
   IO.popen("pr -#{cols} -t", "w"){|io| io.write(items) }
 end
+
+def exec_editor *args
+  editor=ENV['EDITOR']
+  if editor.nil?
+    if system "which -s mate" and $?.success?
+      editor='mate'
+    else
+      editor='vim'
+    end
+  end
+  exec editor, *args
+end