Skip to content

[macOS]Removing Xcode15.4 and its simulators #12177

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

ijunaidm1
Copy link
Contributor

Description

Removing Xcode15.4 and its simulators

Related issue:

Check list

  • Related issue / work item is attached
  • Tests are written (if applicable)
  • Documentation is updated (if applicable)
  • Changes are tested and related VM images are successfully generated

@Copilot Copilot AI review requested due to automatic review settings May 9, 2025 18:08
Copy link
Contributor

@Copilot 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 removes the obsolete Xcode15.4 toolset entries and adds new beta configurations for Xcode 16.4. It updates both the x64 and arm64 JSON configurations to support the new beta version while removing the 15.4 entries.

  • Removed Xcode15.4 toolset entries
  • Added new Xcode16.4 Beta configuration for x64 and arm64
  • Extended arm64 install_runtimes with visionOS support

{ "iOS": ["18.0", "18.1", "18.2", "18.3.1", "18.4", "22F5067a"] },
{ "watchOS": ["11.0", "11.1", "11.2", "11.4", "22T5566a"] },
{ "tvOS": ["18.0", "18.1", "18.2", "18.4", "22L5565a"] },
{ "visionOS": ["2.0", "2.1", "2.2", "2.3", "2.4", "22O5467a"] }
Copy link
Preview

Copilot AI May 9, 2025

Choose a reason for hiding this comment

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

The arm64 install_runtimes array includes a 'visionOS' entry while the x64 configuration does not. If 'visionOS' support is intended for both architectures, consider adding it to the x64 configuration for consistency.

Copilot uses AI. Check for mistakes.

Choose a reason for hiding this comment

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

Suggested change
{ "visionOS": ["2.0", "2.1", "2.2", "2.3", "2.4", "22O5467a"] }
{ "visionOS": ["2.0", "2.1", "2.2", "2.3", "2.4", "22O5467a"] }

Choose a reason for hiding this comment

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

Suggested change
{ "visionOS": ["2.0", "2.1", "2.2", "2.3", "2.4", "22O5467a"] }
{ "visionOS": ["2.0", "2.1", "2.2", "2.3", "2.4", "22O5467a"] }

Comment on lines +7 to +8
"link": "16.4_beta",
"version": "16.4-Beta+16F1t",

Choose a reason for hiding this comment

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

Suggested change
"link": "16.4_beta",
"version": "16.4-Beta+16F1t",
"link": "16.4_Release_Candidate",
"version": "16.4_Release_Candidate+16F6",

Given RC1 for Xcode 16.4 is already out ?

Comment on lines +51 to +52
"link": "16.4_beta",
"version": "16.4-Beta+16F1t",

Choose a reason for hiding this comment

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

Suggested change
"link": "16.4_beta",
"version": "16.4-Beta+16F1t",
"link": "16.4_Release_Candidate",
"version": "16.4_Release_Candidate+16F6",

Given RC1 for Xcode 16.4 is already out ?

@jeslyvarghese
Copy link

@ijunaidm1 The PR seems to have no activity for past 2 weeks, is it abandoned or is planned to be merged soon.
It would be great to have Xcode 16.4 RC1 in either via this PR or another one soon.

@ijunaidm1
Copy link
Contributor Author

@jeslyvarghese - We can close this PR , as we will be working on adding Xcode16.4 RC1 in another PR that is blocked currently for internal technical reasons.

@ijunaidm1
Copy link
Contributor Author

Closing this PR as there is a new PR created/merged for removal of xcode 15.4 . And also we will have a new PR for adding Xcode16.4 release version.

@ijunaidm1 ijunaidm1 closed this May 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants