Skip to content
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

Add visionOS to Package.swift to emit useful compilation errors #398

Merged

Conversation

simonjbeaumont
Copy link
Collaborator

Motivation

We have provision for emitting a useful compilation error when trying to link the generator to an application on darwin platforms, e.g. iOS, tvOS, watchOS. To support this, we add those platforms to the Package.swift so that the code at least tries to compile and then we can emit the #error. This list does not currently include visionOS so users will not get the same useful compilation error.

Modifications

Add visionOS to the platforms in Package.swift. While there, also add this to the platforms in the Package.swift for the client used in the tutorial.

Result

When compiling for visionOS and trying to link the generator, a useful compiler error will be emitted.

Test Plan

Manual.

@simonjbeaumont simonjbeaumont marked this pull request as ready for review November 27, 2023 20:10
@simonjbeaumont simonjbeaumont merged commit 13df787 into apple:main Nov 29, 2023
7 checks passed
@czechboy0 czechboy0 added the 🆕 semver/minor Adds new public API. label Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🆕 semver/minor Adds new public API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants