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

Fix check_coretap_git_origin

Browse Source 
check_coretap_git_origin was not working as intended.
Permit Linuxbrew/homebrew-core as a valid origin.
Factor out check_coretap_git_branch.
This commit is contained in:
Shaun Jackman 2019-01-24 22:51:33 -08:00
parent 2cc50864cc
commit d13a287954
3 changed files with 13 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
Library/Homebrew/diagnostic.rb
View File

@ -581,11 +581,9 @@ module Homebrew
Without a correctly configured origin, Homebrew won't update
properly. You can solve this by adding the Homebrew remote:
git -C "#{coretap_path}" remote add origin #{Formatter.url("https://github.com/Homebrew/homebrew-core.git")}
git -C "#{coretap_path}" remote add origin #{Formatter.url(CoreTap.instance.default_remote)}
EOS
elsif origin !~ %r{Homebrew/homebrew-core(\.git|/)?$}
return if ENV["CI"] && origin.include?("Homebrew/homebrew-test-bot")
elsif origin !~ %r{#{CoreTap.instance.full_name}(\.git|/)?$}
<<~EOS
Suspicious #{CoreTap.instance} git origin remote found.
@ -595,17 +593,22 @@ module Homebrew
Unless you have compelling reasons, consider setting the
origin remote to point at the main repository by running:
git -C "#{coretap_path}" remote set-url origin #{Formatter.url("https://github.com/Homebrew/homebrew-core.git")}
git -C "#{coretap_path}" remote set-url origin #{Formatter.url(CoreTap.instance.default_remote)}
EOS
end
end
def check_coretap_git_branch
return if ENV["CI"]
coretap_path = CoreTap.instance.path
return if !Utils.git_available? || !(coretap_path/".git").exist?
branch = coretap_path.git_branch
return if branch.nil? || branch =~ /master/
<<~EOS
Homebrew/homebrew-core is not on the master branch
#{CoreTap.instance.full_name} is not on the master branch
Check out the master branch by running:
git -C "$(brew --repo homebrew/core)" checkout master

10
Library/Homebrew/extend/os/linux/tap.rb
View File

@ -1,9 +1,7 @@
class CoreTap < Tap
def default_remote
if ENV["HOMEBREW_FORCE_HOMEBREW_ON_LINUX"]
"https://github.com/Homebrew/homebrew-core".freeze
else
"https://github.com/Linuxbrew/homebrew-core".freeze
end
# @private
def initialize
super "Homebrew", "core"
@full_name = "Linuxbrew/homebrew-core" unless ENV["HOMEBREW_FORCE_HOMEBREW_ON_LINUX"]
end
end

4
Library/Homebrew/tap.rb
View File

@ -635,10 +635,6 @@ end
# A specialized {Tap} class for the core formulae.
class CoreTap < Tap
def default_remote
"https://github.com/Homebrew/homebrew-core".freeze
end
# @private
def initialize
super "Homebrew", "core"