Skip to content

Mark Platform.openbsd as available since 5.6 #3970

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

Conversation

abertelrud
Copy link
Contributor

Mark Platform.openbsd as available since 5.6.

Motivation:

Allows this new enum to be used in packages marked with 5.6.

Modifications:

  • Change 999.0 to 5.6 availability and add a CHANGELOG entry.

This will also be nominated to the release/5.6 branch.

@abertelrud abertelrud self-assigned this Jan 4, 2022
@abertelrud abertelrud added the ready Author believes the PR is ready to be merged & any feedback has been addressed label Jan 4, 2022
@abertelrud
Copy link
Contributor Author

@swift-ci smoke test

tomerd
tomerd previously approved these changes Jan 4, 2022
Copy link
Contributor

@tomerd tomerd left a comment

Choose a reason for hiding this comment

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

what level of support do we offer for this platform? is actually fully supported?

@tomerd tomerd self-requested a review January 4, 2022 18:23
Copy link
Contributor

@tomerd tomerd left a comment

Choose a reason for hiding this comment

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

what level of support do we offer for this platform? is actually fully supported?

@tomerd tomerd dismissed their stale review January 4, 2022 18:24

made by mistake

@abertelrud
Copy link
Contributor Author

That's a good question — I will take a look at the original PR. I just noticed it as being 999.0 when changing the command plugin APIs, but we should leave as 999.0 if there isn't actually any support.

@abertelrud
Copy link
Contributor Author

Looks like this was introduced in #3572, where the statement is that SwiftPM now builds on OpenBSD. But since that's not an officially supported platform AFAIK, probably the right thing is to leave it as an experimental feature at 999.0 (which means SwiftPM 5.6 won't support it, but main toolchains will).

@abertelrud abertelrud added DO NOT MERGE and removed ready Author believes the PR is ready to be merged & any feedback has been addressed labels Jan 5, 2022
@abertelrud
Copy link
Contributor Author

Closing this PR, leaving the annotation as 999.0.

@abertelrud abertelrud closed this Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants