From fded4d0385171877bc6e60a2f265a2095be652da Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Sun, 3 Aug 2014 10:47:47 -0500 Subject: Remove an is_a check --- Library/Homebrew/extend/ENV/shared.rb | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'Library/Homebrew/extend') 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 -- cgit v1.2.3