From 708935d2c8adf659095a27766ce8ea58832a52fe Mon Sep 17 00:00:00 2001 From: Carlo Cabrera <30379873+carlocab@users.noreply.github.com> Date: Tue, 6 Sep 2022 18:35:34 +0800 Subject: [PATCH] cmd/fetch: rename `fetch_failed` to `fetch_tries`. This makes the code substantially easier to understand. Follow-up to #13807. --- Library/Homebrew/cmd/fetch.rb | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Library/Homebrew/cmd/fetch.rb b/Library/Homebrew/cmd/fetch.rb index 465347136a..5bbe534ee7 100644 --- a/Library/Homebrew/cmd/fetch.rb +++ b/Library/Homebrew/cmd/fetch.rb @@ -162,17 +162,17 @@ module Homebrew end def retry_fetch?(f, args:) - @fetch_failed ||= Hash.new { |h, k| h[k] = 1 } - if args.retry? && (@fetch_failed[f] < FETCH_MAX_TRIES) - wait = 2 ** @fetch_failed[f] - remaining = FETCH_MAX_TRIES - @fetch_failed[f] + @fetch_tries ||= Hash.new { |h, k| h[k] = 1 } + if args.retry? && (@fetch_tries[f] < FETCH_MAX_TRIES) + wait = 2 ** @fetch_tries[f] + remaining = FETCH_MAX_TRIES - @fetch_tries[f] what = "try".pluralize(remaining) ohai "Retrying download in #{wait}s... (#{remaining} #{what} left)" sleep wait f.clear_cache - @fetch_failed[f] += 1 + @fetch_tries[f] += 1 true else Homebrew.failed = true