From c6d75bfcec5b953a3c2ded37212b545304f4e35c Mon Sep 17 00:00:00 2001
From: Jack Nagel <jacknagel@gmail.com>
Date: Mon, 16 Jun 2014 13:38:14 -0500
Subject: [PATCH] Move head-only checks to cmd/install

Fixes Homebrew/homebrew#30084.
Closes Homebrew/homebrew#30203.
---
 Library/Homebrew/cmd/install.rb       | 15 +++++++++++++++
 Library/Homebrew/formula_installer.rb | 13 -------------
 2 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/Library/Homebrew/cmd/install.rb b/Library/Homebrew/cmd/install.rb
index d975f485c2..ca228e3387 100644
--- a/Library/Homebrew/cmd/install.rb
+++ b/Library/Homebrew/cmd/install.rb
@@ -24,6 +24,21 @@ module Homebrew extend self
       end
     end unless ARGV.force?
 
+    ARGV.formulae.each do |f|
+      # Building head-only without --HEAD is an error
+      if not ARGV.build_head? and f.stable.nil?
+        raise CannotInstallFormulaError, <<-EOS.undent
+        #{f} is a head-only formula
+        Install with `brew install --HEAD #{f.name}`
+        EOS
+      end
+
+      # Building stable-only with --HEAD is an error
+      if ARGV.build_head? and f.head.nil?
+        raise CannotInstallFormulaError, "No head is defined for #{f.name}"
+      end
+    end
+
     perform_preinstall_checks
 
     begin
diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb
index 7321f9c4de..ea853be29e 100644
--- a/Library/Homebrew/formula_installer.rb
+++ b/Library/Homebrew/formula_installer.rb
@@ -109,19 +109,6 @@ class FormulaInstaller
       raise FormulaAlreadyInstalledError, msg
     end
 
-    # Building head-only without --HEAD is an error
-    if not ARGV.build_head? and f.stable.nil?
-      raise CannotInstallFormulaError, <<-EOS.undent
-        #{f} is a head-only formula
-        Install with `brew install --HEAD #{f.name}
-      EOS
-    end
-
-    # Building stable-only with --HEAD is an error
-    if ARGV.build_head? and f.head.nil?
-      raise CannotInstallFormulaError, "No head is defined for #{f.name}"
-    end
-
     unless ignore_deps?
       unlinked_deps = f.recursive_dependencies.map(&:to_formula).select do |dep|
         dep.installed? and not dep.keg_only? and not dep.linked_keg.directory?