From a196898ad31cc2743cfafa5262d1d6f9e945e5cc Mon Sep 17 00:00:00 2001 From: Sam Ford <1584702+samford@users.noreply.github.com> Date: Sun, 28 Jul 2024 10:17:23 -0400 Subject: [PATCH] unpack_strategy/cab: align method order This simply moves the `Cab#dependencies` method above `#extract_to_dir`, to align it with the other strategies. --- Library/Homebrew/unpack_strategy/cab.rb | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Library/Homebrew/unpack_strategy/cab.rb b/Library/Homebrew/unpack_strategy/cab.rb index c9427f86fc..cb3aa57ee9 100644 --- a/Library/Homebrew/unpack_strategy/cab.rb +++ b/Library/Homebrew/unpack_strategy/cab.rb @@ -16,6 +16,11 @@ module UnpackStrategy path.magic_number.match?(/\AMSCF/n) end + sig { returns(T::Array[Formula]) } + def dependencies + @dependencies ||= T.let([Formula["cabextract"]], T.nilable(T::Array[Formula])) + end + sig { override.params(unpack_dir: Pathname, basename: Pathname, verbose: T::Boolean).void } def extract_to_dir(unpack_dir, basename:, verbose:) system_command! "cabextract", @@ -23,10 +28,5 @@ module UnpackStrategy env: { "PATH" => PATH.new(Formula["cabextract"].opt_bin, ENV.fetch("PATH")) }, verbose: end - - sig { returns(T::Array[Formula]) } - def dependencies - @dependencies ||= T.let([Formula["cabextract"]], T.nilable(T::Array[Formula])) - end end end