diff options
| author | Mike McQuaid | 2016-11-15 15:29:03 +0000 |
|---|---|---|
| committer | GitHub | 2016-11-15 15:29:03 +0000 |
| commit | a92da096091729f4df8a03b16515a6441bb00c08 (patch) | |
| tree | d92e1d9f08cc4024963f9603236500ed8bdf1472 /Library/Homebrew | |
| parent | 673c4c78bdb9f5ae2be47755e66863f2963c4f1a (diff) | |
| parent | 727263e90620b1edabffdb6444018d08f8c43fda (diff) | |
| download | brew-a92da096091729f4df8a03b16515a6441bb00c08.tar.bz2 | |
Merge pull request #1509 from MikeMcQuaid/development-tools-no-nil-versions
development_tools: don't create Versions from `nil`
Diffstat (limited to 'Library/Homebrew')
| -rw-r--r-- | Library/Homebrew/development_tools.rb | 25 |
1 files changed, 15 insertions, 10 deletions
diff --git a/Library/Homebrew/development_tools.rb b/Library/Homebrew/development_tools.rb index febe04b21..62f9b3d71 100644 --- a/Library/Homebrew/development_tools.rb +++ b/Library/Homebrew/development_tools.rb @@ -43,8 +43,9 @@ class DevelopmentTools def gcc_40_build_version @gcc_40_build_version ||= - if (path = locate("gcc-4.0")) - Version.new `#{path} --version 2>/dev/null`[/build (\d{4,})/, 1].to_i + if (path = locate("gcc-4.0")) && + build_version = `#{path} --version 2>/dev/null`[/build (\d{4,})/, 1] + Version.new build_version else Version::NULL end @@ -55,8 +56,9 @@ class DevelopmentTools @gcc_42_build_version ||= begin gcc = locate("gcc-4.2") || HOMEBREW_PREFIX.join("opt/apple-gcc42/bin/gcc-4.2") - if gcc.exist? && !gcc.realpath.basename.to_s.start_with?("llvm") - Version.new `#{gcc} --version 2>/dev/null`[/build (\d{4,})/, 1] + if gcc.exist? && !gcc.realpath.basename.to_s.start_with?("llvm")&& + build_version = `#{gcc} --version 2>/dev/null`[/build (\d{4,})/, 1] + Version.new build_version else Version::NULL end @@ -66,8 +68,9 @@ class DevelopmentTools def clang_version @clang_version ||= - if (path = locate("clang")) - Version.new `#{path} --version`[/(?:clang|LLVM) version (\d\.\d)/, 1] + if (path = locate("clang")) && + build_version = `#{path} --version`[/(?:clang|LLVM) version (\d\.\d)/, 1] + Version.new build_version else Version::NULL end @@ -75,8 +78,9 @@ class DevelopmentTools def clang_build_version @clang_build_version ||= - if (path = locate("clang")) - Version.new `#{path} --version`[/clang-(\d{2,})/, 1] + if (path = locate("clang")) && + build_version = `#{path} --version`[/clang-(\d{2,})/, 1] + Version.new build_version else Version::NULL end @@ -86,8 +90,9 @@ class DevelopmentTools (@non_apple_gcc_version ||= {}).fetch(cc) do path = HOMEBREW_PREFIX.join("opt", "gcc", "bin", cc) path = locate(cc) unless path.exist? - version = if path - Version.new(`#{path} --version`[/gcc(?:-\d(?:\.\d)? \(.+\))? (\d\.\d\.\d)/, 1]) + version = if path && + build_version = `#{path} --version`[/gcc(?:-\d(?:\.\d)? \(.+\))? (\d\.\d\.\d)/, 1] + Version.new build_version else Version::NULL end |