Skip to content

[ci] Try better fix for iOS simulator (#32108)#32124

Merged
rmarinho merged 1 commit intorelease/10.0.1xxfrom
update-release-simulator
Oct 22, 2025
Merged

[ci] Try better fix for iOS simulator (#32108)#32124
rmarinho merged 1 commit intorelease/10.0.1xxfrom
update-release-simulator

Conversation

@rmarinho
Copy link
Member

Description of Change

Updates for simulator install

* tweak

* delete all simulator runtimes if failing to install one

---------

Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>
# Conflicts:
#	eng/pipelines/common/provision.yml
Copilot AI review requested due to automatic review settings October 21, 2025 21:27
@rmarinho rmarinho added this to the .NET 10.0 GA milestone Oct 21, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the iOS simulator provisioning logic in the CI pipeline to address compatibility issues with Xcode 26.0. The key change is adding conditional logic to explicitly install the iOS 26.0 simulator with the universal architecture variant when using Xcode 26.0.x, as the iOS 26.0.1 simulator doesn't work correctly. It also includes a retry mechanism that deletes all simulator runtimes if the initial installation fails.

  • Adds conditional check for Xcode 26.0.x versions
  • Implements retry logic with simulator runtime cleanup on installation failure
  • Explicitly specifies universal architecture variant for broader simulator compatibility

Comment on lines +100 to +101
echo " still $C simulators left..."
break
Copy link

Copilot AI Oct 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The condition logic is inverted. The message 'still $C simulators left...' is printed when C equals 0 (no simulators remaining), but it should be printed when C is greater than 0. The break statement should execute when C equals 0, and the echo should execute when C is greater than 0.

Suggested change
echo " still $C simulators left..."
break
break
else
echo " still $C simulators left..."

Copilot uses AI. Check for mistakes.
@rmarinho rmarinho merged commit 84a6cbe into release/10.0.1xx Oct 22, 2025
20 of 24 checks passed
@rmarinho rmarinho deleted the update-release-simulator branch October 22, 2025 08:40
@github-actions github-actions bot locked and limited conversation to collaborators Nov 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants