From f405f7316c328f561afcb72613c1cad47663292b Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Fri, 1 Aug 2025 08:47:38 +0100 Subject: [PATCH] extend/kernel: fix odisabled call. This must set `disable` to `true` to have correct behaviour. Also, don't allow setting it to `false` to avoid confusion as that's what `odeprecated` is for. --- Library/Homebrew/extend/kernel.rb | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/Library/Homebrew/extend/kernel.rb b/Library/Homebrew/extend/kernel.rb index 39e01f6b75..cbcbc8fefe 100644 --- a/Library/Homebrew/extend/kernel.rb +++ b/Library/Homebrew/extend/kernel.rb @@ -223,16 +223,15 @@ module Kernel end sig { - params(method: String, replacement: T.nilable(T.any(String, Symbol)), disable: T::Boolean, + params(method: String, replacement: T.nilable(T.any(String, Symbol)), disable_on: T.nilable(Time), disable_for_developers: T::Boolean, caller: T::Array[String]).void } def odisabled(method, replacement = nil, - disable: false, disable_on: nil, disable_for_developers: true, caller: send(:caller)) # This odeprecated should stick around indefinitely. - odeprecated(method, replacement, disable:, disable_on:, disable_for_developers:, caller:) + odeprecated(method, replacement, disable: true, disable_on:, disable_for_developers:, caller:) end sig { params(formula: T.any(String, Formula)).returns(String) }