From 28c593ed6ac0013fae888e69010b7da6b2e8d2d6 Mon Sep 17 00:00:00 2001 From: Carlo Cabrera <30379873+carlocab@users.noreply.github.com> Date: Mon, 15 May 2023 13:44:51 +0800 Subject: [PATCH] simulate_system: fix `tap_syntax` job errors Fixes Error: Cannot simulate :ventura :intel while already simulating :macos. See https://github.com/Homebrew/homebrew-core/actions/runs/4974116510/jobs/8900390980#step:4:24. --- Library/Homebrew/simulate_system.rb | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/Library/Homebrew/simulate_system.rb b/Library/Homebrew/simulate_system.rb index 6de2e98241..2b1b0faa01 100644 --- a/Library/Homebrew/simulate_system.rb +++ b/Library/Homebrew/simulate_system.rb @@ -19,10 +19,8 @@ module Homebrew def with(os: T.unsafe(nil), arch: T.unsafe(nil), &_block) raise ArgumentError, "At least one of `os` or `arch` must be specified." if !os && !arch - if self.os || self.arch - raise "Cannot simulate#{os&.inspect&.prepend(" ")}#{arch&.inspect&.prepend(" ")} while already " \ - "simulating#{self.os&.inspect&.prepend(" ")}#{self.arch&.inspect&.prepend(" ")}." - end + old_os = self.os + old_arch = self.arch begin self.os = os if os @@ -30,7 +28,8 @@ module Homebrew yield ensure - clear + @os = old_os + @arch = old_arch end end