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
Transition away from Foundation.URL
#6706
Conversation
Saw a couple test failures locally, so this isn't really ready to land, yet. We will need to cherry-pick this into 5.9 to be compatible with Sonoma there. I believe older Xcodes are already not supported on Sonoma, which is good, but I wonder whether it is worth having some kind of warning in 5.8 which tells people that it doesn't work on Sonoma if we're shipping more point updates. |
@swift-ci please smoke test |
That one looks scary 🤣 |
Ah, looks like that is an issue with the |
@swift-ci please smoke test |
@swift-ci please smoke test windows |
I think the remaining test failure is from me accidentally breaking V1 of fingerprint storage. |
@swift-ci please smoke test |
@swift-ci please smoke test windows |
I think this should be working now, so it can come out of draft. Note that I haven't tested this on Sonoma yet, it is just based on code inspection to identify APIs that need to change, there might be more required here. |
Looks like the CMake build is broken, I think I forgot to add the new file there. |
@swift-ci please smoke test |
@swift-ci please smoke test windows |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some small comments / ideas
f56d89c
to
36e395a
Compare
Foundation.URL
Foundation.URL
@swift-ci please smoke test |
@swift-ci please smoke test windows |
Had to switch over some more APIs to make the tests actually pass on Sonoma. |
@swift-ci please smoke test windows |
28055d2
to
930c0d2
Compare
@swift-ci please smoke test |
@swift-ci please smoke test windows |
No Windows CI right now, I guess... |
I changed the CMake config, but didn't actually rename the file 😅 |
The parser of `NSURL` is changing in macOS Sonoma and will no longer be compatible with the GitHub-style SSH URLs which means we have to transition back to using our own URL type (which is a wrapper of `String` for now) in order to continue to support SSH URLs. rdar://112482783
930c0d2
to
0010bc3
Compare
@swift-ci please smoke test |
@swift-ci please smoke test windows |
The parser of
NSURL
is changing in macOS Sonoma and will no longer be compatible with the GitHub-style SSH URLs which means we have to transition back to using our own URL type (which is a wrapper ofString
for now) in order to continue to support SSH URLs.rdar://112482783