Skip to content

Conversation

@grynspan
Copy link
Contributor

@grynspan grynspan commented Nov 6, 2025

This PR removes a workaround for the lack of #embed support in OpenBSD 7.7's version of clang. OpenBSD 7.8 has a newer clang that has full #embed support, so we no longer need to special-case OpenBSD here.

Swift Testing will continue to compile for OpenBSD 7.7, but will emit a compile-time warning of the form:

SWT_TESTING_LIBRARY_VERSION not defined and could not read from VERSION.txt at
compile time: testing library version is unavailable

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

This PR removes a workaround for the lack of `#embed` support in OpenBSD 7.7's
version of clang. OpenBSD 7.8 has a newer clang that has full `#embed` support,
so we no longer need to special-case OpenBSD here.

Swift Testing will continue to compile for OpenBSD 7.7, but will emit a
compile-time warning of the form:

> SWT_TESTING_LIBRARY_VERSION not defined and could not read from VERSION.txt at
> compile time: testing library version is unavailable
@grynspan grynspan added this to the Swift 6.x (main) milestone Nov 6, 2025
@grynspan grynspan self-assigned this Nov 6, 2025
@grynspan grynspan added the workaround Workaround for an issue in another component (may need to revert later) label Nov 6, 2025
@grynspan grynspan added openbsd 🐡 OpenBSD support tech-debt 💾 reduces technical debt labels Nov 6, 2025
@grynspan
Copy link
Contributor Author

grynspan commented Nov 6, 2025

@3405691582 FYI.

@grynspan grynspan merged commit db33906 into main Nov 6, 2025
27 checks passed
@grynspan grynspan deleted the jgrynspan/remove-openbsd7.7-support branch November 6, 2025 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

openbsd 🐡 OpenBSD support tech-debt 💾 reduces technical debt workaround Workaround for an issue in another component (may need to revert later)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants