Skip to content

Conversation

@gjcairo
Copy link
Collaborator

@gjcairo gjcairo commented Aug 4, 2025

No description provided.

@gjcairo gjcairo merged commit ff5e6cf into main Aug 4, 2025
@gjcairo gjcairo deleted the new-product branch August 4, 2025 15:12

let package = Package(
name: "HTTPServer",
platforms: [.macOS("16")],

Choose a reason for hiding this comment

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

Won't the removal of the minimum version break the build?

Copy link
Contributor

Choose a reason for hiding this comment

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

Everything should be properly availability annotated IIRC

Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if I landed that though. @gjcairo you can check

Choose a reason for hiding this comment

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

I don't see it eg in https://github.com/swift-server/swift-http-server/blob/main/Sources/HTTPServer/HTTPServer.swift, so I believe this won't currently build on main.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah this builds fine locally on my Mac, but fails on 6.1. There are annotations missing around Span. I was expecting some warnings to show up in Xcode but nope. I'll fix this up and get CI set up.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ah, there was an unsafe flag disabling the checks on Xcode :D I'll get rid of that too

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.

4 participants