Skip to content
Snippets Groups Projects
Unverified Commit 64816651 authored by Rylan Polster's avatar Rylan Polster
Browse files

Only tap homebrew/cask on CaskUnavailableError in brew install

parent ebba3698
No related branches found
No related tags found
No related merge requests found
......@@ -99,8 +99,6 @@ module Homebrew
begin
return Cask::CaskLoader.load(name, config: Cask::Config.from_args(@parent))
rescue Cask::CaskUnavailableError => e
retry if Tap.install_default_cask_tap_if_necessary
raise e if only == :cask
end
end
......
......@@ -153,8 +153,14 @@ module Homebrew
EOS
end
formulae, casks = args.named.to_formulae_and_casks
.partition { |formula_or_cask| formula_or_cask.is_a?(Formula) }
begin
formulae, casks = args.named.to_formulae_and_casks
.partition { |formula_or_cask| formula_or_cask.is_a?(Formula) }
rescue FormulaOrCaskUnavailableError, Cask::CaskUnavailableError => e
retry if Tap.install_default_cask_tap_if_necessary(force: args.cask?)
raise e
end
if casks.any?
Cask::Cmd::Install.install_casks(
......
......@@ -2,10 +2,10 @@
# frozen_string_literal: true
class Tap
def self.install_default_cask_tap_if_necessary
def self.install_default_cask_tap_if_necessary(force: false)
return false if default_cask_tap.installed?
return false if Tap.untapped_official_taps.include?(default_cask_tap.name)
return false if !force && Tap.untapped_official_taps.include?(default_cask_tap.name)
default_cask_tap.install
true
......
......@@ -67,8 +67,8 @@ class Tap
@default_cask_tap ||= fetch("Homebrew", "cask")
end
sig { returns(T::Boolean) }
def self.install_default_cask_tap_if_necessary
sig { params(force: T::Boolean).returns(T::Boolean) }
def self.install_default_cask_tap_if_necessary(force: false)
false
end
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment