Skip to content

Ensure proper method(s) are supportive of cross-platform swift 6 support #108

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

Closed
wants to merge 11 commits into from

Conversation

furby-tm
Copy link
Contributor

@furby-tm furby-tm commented Mar 2, 2025

No description provided.

furby-tm added 11 commits March 1, 2025 01:34
Signed-off-by: furby™ <devs@wabi.foundation>
Signed-off-by: furby™ <devs@wabi.foundation>
Signed-off-by: furby™ <devs@wabi.foundation>
Signed-off-by: furby™ <devs@wabi.foundation>
* Part of our efforts to condense and streamline all the various C/C++ dependencies under one central repository, this is important because Swift makes it all too easy to have 30 different zlib libraries for example, which leads to a headache of duplicated symbols and what not.
* That central repository is https://github.com/the-swift-collective

Signed-off-by: furby™ <devs@wabi.foundation>
Signed-off-by: furby™ <devs@wabi.foundation>
Signed-off-by: furby™ <devs@wabi.foundation>
Signed-off-by: furby™ <devs@wabi.foundation>
* This ensures scui works with swift 6 strict concurrency - however it is lopsided in that makeCoordinator() is not marked with a main actor.

Signed-off-by: furby™ <devs@wabi.foundation>
Signed-off-by: furby™ <devs@wabi.foundation>
* The appropriate method that should NOT have a main actor is
  determineViewSize, since that will break compiling with swift 6
  strict concurrency.

Signed-off-by: furby™ <devs@wabi.foundation>
@stackotter
Copy link
Owner

For future readers, I've just done the final change manually (this is #105 + one more change).

@stackotter stackotter closed this Mar 2, 2025
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.

2 participants