mirrors/brew
1
0
Fork 0
mirror of https://github.com/Homebrew/brew.git synced 2025-07-14 16:09:03 +08:00
Code Issues Packages Projects Releases Wiki Activity

development_tools: don't create Versions from nil

Browse Source 
This cause issues when e.g. using `debrew.rb` on a failing `system`
command in a formula.
This commit is contained in:
Mike McQuaid 2016-11-15 14:43:56 +00:00
parent c1af8fba55
commit 727263e906
1 changed files with 15 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
Library/Homebrew/development_tools.rb
View File

@ -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