Merge pull request #13718 from Rylan12/fix-bump-cask-pr
Fix multi-arch cask `sha256` updates
This commit is contained in:
commit
c84453b152
@ -138,13 +138,13 @@ module Homebrew
|
||||
replacement_pairs << fetch_cask(tmp_contents, config: lang_config)
|
||||
end
|
||||
|
||||
# TODO: Use SimulateSystem once all casks use on_system blocks
|
||||
if tmp_contents.include?("Hardware::CPU.intel?")
|
||||
other_intel = !Hardware::CPU.intel?
|
||||
Homebrew::SimulateSystem.arch = other_intel ? :intel : :arm
|
||||
other_contents = tmp_contents.gsub("Hardware::CPU.intel?", other_intel.to_s)
|
||||
other_cask = Cask::CaskLoader.load(other_contents)
|
||||
# TODO: remove the `Hardware::CPU.intel?` substitution once no casks use the conditional
|
||||
other_intel = !Hardware::CPU.intel?
|
||||
Homebrew::SimulateSystem.arch = other_intel ? :intel : :arm
|
||||
other_contents = tmp_contents.gsub("Hardware::CPU.intel?", other_intel.to_s)
|
||||
other_cask = Cask::CaskLoader.load(other_contents)
|
||||
|
||||
if other_cask.url.to_s != tmp_cask.url.to_s
|
||||
if other_cask.sha256 != :no_check && other_cask.language.blank?
|
||||
replacement_pairs << fetch_cask(other_contents)
|
||||
end
|
||||
@ -153,9 +153,9 @@ module Homebrew
|
||||
lang_config = other_cask.config.merge(Cask::Config.new(explicit: { languages: [language] }))
|
||||
replacement_pairs << fetch_cask(other_contents, config: lang_config)
|
||||
end
|
||||
|
||||
Homebrew::SimulateSystem.clear
|
||||
end
|
||||
|
||||
Homebrew::SimulateSystem.clear
|
||||
end
|
||||
end
|
||||
|
||||
@ -163,8 +163,8 @@ module Homebrew
|
||||
hash_regex = (old_hash == :no_check) ? ":no_check" : "[\"']#{Regexp.escape(old_hash.to_s)}[\"']"
|
||||
|
||||
replacement_pairs << [
|
||||
/sha256\s+#{hash_regex}/m,
|
||||
"sha256 #{(new_hash == :no_check) ? ":no_check" : "\"#{new_hash}\""}",
|
||||
/#{hash_regex}/m,
|
||||
((new_hash == :no_check) ? ":no_check" : "\"#{new_hash}\"").to_s,
|
||||
]
|
||||
end
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user