From 76a54d1455e76b0eb201c64dae5527458ef8588c Mon Sep 17 00:00:00 2001 From: Sam Ford <1584702+samford@users.noreply.github.com> Date: Wed, 13 Aug 2025 09:59:18 -0400 Subject: [PATCH] bump-cask-pr: ensure new_cask has a url We sometimes see errors like "attempted to use a `Downloadable` without a URL!" in the homebrew/cask autobump workflow log because `bump-cask-pr` can simulate Linux even if a cask doesn't support it, leading to this error. This is something that should be resolved in the future once I finally wrap up my related work to detect OS/arch requirements but this adds a simple guard to address this in the interim time. --- Library/Homebrew/dev-cmd/bump-cask-pr.rb | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Library/Homebrew/dev-cmd/bump-cask-pr.rb b/Library/Homebrew/dev-cmd/bump-cask-pr.rb index e7ce29cc99..bf9d9aafe5 100644 --- a/Library/Homebrew/dev-cmd/bump-cask-pr.rb +++ b/Library/Homebrew/dev-cmd/bump-cask-pr.rb @@ -273,6 +273,8 @@ module Homebrew end languages.each do |language| new_cask = Cask::CaskLoader.load(tmp_contents) + next unless new_cask.url + new_cask.config = if language.blank? tmp_cask.config else