Skip to content

2.x: Support index-import on all Xcode 16.x versions #1505

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

Merged

Conversation

luispadron
Copy link
Contributor

@luispadron luispadron commented Apr 1, 2025

This PR updates the index-import dependency to include both version 5.8 and 6.1 as the hash algorithm changed in Swift 6.1.
To make this change backwards compatible we switch to the 5.8 version on Xcode 16.2 and under.

@luispadron luispadron force-pushed the luis/update-hash-algorithm-for-xcode-16.3-rules_swift-2.x branch from c5400d3 to 33b84dd Compare April 1, 2025 20:06
This PR updates the index-import dependency to include both version 5.8 and 6.1 as the hash algorithm changed in Swift 6.1.
To make this change backwards compatible we switch to the 5.8 version on Xcode 16.2 and under.
@luispadron luispadron force-pushed the luis/update-hash-algorithm-for-xcode-16.3-rules_swift-2.x branch from 33b84dd to 49306d7 Compare April 3, 2025 17:44
@luispadron luispadron changed the title 2.x: update hash algorithm for Xcode 16.3 2.x: Support index-import on all Xcode 16.x versions Apr 3, 2025
@luispadron luispadron marked this pull request as ready for review April 3, 2025 17:48
@luispadron
Copy link
Contributor Author

@keith im actually going to drop the apple_support update from the 2.x branch unless you think its fine to drop support for Bazel 6 like this?

@keith
Copy link
Member

keith commented Apr 3, 2025

yea sounds fine, this branch doesn't need to support last_green

@keith
Copy link
Member

keith commented Apr 3, 2025

you can remove that ci config for this branch instead

@luispadron luispadron force-pushed the luis/update-hash-algorithm-for-xcode-16.3-rules_swift-2.x branch from 56a035e to dbde6f7 Compare April 3, 2025 18:01
@luispadron luispadron enabled auto-merge (squash) April 3, 2025 18:01
@luispadron luispadron merged commit 859afa8 into 2.x Apr 3, 2025
12 checks passed
@luispadron luispadron deleted the luis/update-hash-algorithm-for-xcode-16.3-rules_swift-2.x branch April 3, 2025 18:03
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.

2 participants