admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

OS::Mac::sdk: remove ensure block

Browse Source 
Don't use ensure to do the final check because it won't affect the
return value of the overall function.

Thanks to @xucheng for catching this bug.

Closes Homebrew/homebrew#50536.

Signed-off-by: ilovezfs <ilovezfs@icloud.com>
This commit is contained in:
ilovezfs 2016-03-30 04:03:19 -07:00 committed by Xu Cheng
parent 45e138ffc6
commit 7651757709
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Library/Homebrew/os/mac.rb
View File

@ -100,10 +100,9 @@ module OS
end
rescue SDKLocator::NoSDKError
sdk = @locator.latest_sdk
ensure
# only return an SDK older than the OS version if it was specifically requested
sdk if v || (!sdk.nil? && sdk.version >= version)
end
# Only return an SDK older than the OS version if it was specifically requested
sdk if v || (!sdk.nil? && sdk.version >= version)
end
# Returns the path to an SDK or nil, following the rules set by #sdk.