Skip to content
This repository was archived by the owner on Jun 1, 2023. It is now read-only.

swift-doc: load FoundationNetworking on non-Darwin platforms #257

Closed
wants to merge 3 commits into from

Conversation

compnerd
Copy link
Contributor

Invert the condition to not link against Foundation networking only on
non-Darwin platforms. Every other platform uses
swift-corelibs-foundation which provides a FoundationNetworking which
must be linked against in order to have the networking functionality
work. This repairs the fetch of the CSS for Windows hosts.

Invert the condition to not link against Foundation networking only on
non-Darwin platforms.  Every other platform uses
swift-corelibs-foundation which provides a FoundationNetworking which
must be linked against in order to have the networking functionality
work.  This repairs the fetch of the CSS for Windows hosts.
Copy link
Contributor

@MaxDesiatov MaxDesiatov left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@compnerd
Copy link
Contributor Author

Hmm, what is with the signed commit requirement on this repository? I don't think that I have a setup where configuring that is convenient. I suppose that I could do this through the web editor :/

@MaxDesiatov
Copy link
Contributor

@mattt I would be happy to merge, but the signing requirement prevents me from doing it and I can't override it. I think if you'd like all commits in master to be signed, enabling "Squash and Merge" mode as the default should be enough. PRs merged that way are automatically signed by GitHub.

@compnerd
Copy link
Contributor Author

@MaxDesiatov I'm going to close this as I've opened #265 via the web editor which should allow me to merge. Would you mind approving that?

@compnerd compnerd closed this Apr 28, 2021
@compnerd compnerd deleted the foundations branch April 28, 2021 15:47
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.

3 participants