Remove an is_a check

1 files changed, 7 insertions, 1 deletions

diff --git a/Library/Homebrew/extend/ENV/shared.rb b/Library/Homebrew/extend/ENV/shared.rb
index 583c3c639..3e95a4dc5 100644
--- a/Library/Homebrew/extend/ENV/shared.rb
+++ b/Library/Homebrew/extend/ENV/shared.rb
@@ -130,7 +130,13 @@ module SharedEnvExtension
   # an alternate compiler, altering the value of environment variables.
   # If no valid compiler is found, raises an exception.
   def validate_cc!(formula)
-    if formula.fails_with? compiler
+    # FIXME
+    # The compiler object we pass to fails_with? has no version information
+    # attached to it. This means that if we pass Compiler.new(:clang), the
+    # selector will be invoked if the formula fails with any version of clang.
+    # I think we can safely remove this conditional and always invoke the
+    # selector.
+    if formula.fails_with? Compiler.new(compiler)
       send CompilerSelector.new(formula).compiler
     end
   end