Skip to content

Conversation

@dreampiggy
Copy link
Contributor

@dreampiggy dreampiggy commented Jun 22, 2023

New Pull Request Checklist

  • I have read and understood the CONTRIBUTING guide

  • I have read the Documentation

  • I have searched for a similar pull request in the project and found none

  • I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

  • I have added the required tests to prove the fix/feature I am adding

  • I have updated the documentation (if necessary)

  • I have run the tests and they pass

  • I have run the lint and it passes (pod lib lint)

This merge request fixes / refers to the following issues: ...

Pull Request Description

See:

https://twitter.com/DreamingPiggy/status/1671841279205076992?s=20

@scottsykora
Copy link

Looks like you need to use TARGET_OS_VISION instead of TARGET_OS_XR in the latest betas.

@dreampiggy
Copy link
Contributor Author

dreampiggy commented Sep 2, 2023

Recently I still has no much more time on SDWebImage open source project.

So I deceide to separete the visionOS into 2 parts.

  1. This PR, only contains the Source Code changes with macro detection, to make at least some user who can build SDWebImage for visionOS using the sour code
  2. Another PR, which contains the unit tests, demo, package manager integration (CocoaPods/Carthage/SPM)

The part 1 will merged into 5.x, which does not require you to use Xcode 15, so you can still use Xcode 14/13 to build SDWebImage. For 6.0.0, Xcode 15 is required

@dreampiggy dreampiggy changed the title WIP: Support visionOS (xrOS) (Part 1) Support visionOS (xrOS) in source code level (no package manager support) Sep 2, 2023
@dreampiggy dreampiggy modified the milestones: 6.0.0, 5.18.0 Sep 2, 2023
The scale using traitCollection.displayScale
This only change source code
@dreampiggy
Copy link
Contributor Author

Tested can be compiled on visionOS 1.0 with Xcode 15 Beta 7

截屏2023-09-02 20 13 13

And, does not break the exists Xcode 14 toolchain users

@dreampiggy dreampiggy changed the title (Part 1) Support visionOS (xrOS) in source code level (no package manager support) (Part 1) Support to build for visionOS (no package manager support) Sep 2, 2023
@dreampiggy dreampiggy added the visionOS Apple Vision label Sep 2, 2023
@dreampiggy dreampiggy merged commit 82cfc45 into SDWebImage:master Sep 2, 2023
@dreampiggy dreampiggy deleted the feature/visionOS branch September 2, 2023 12:19
@dreampiggy
Copy link
Contributor Author

dreampiggy commented Sep 2, 2023

available in 5.18.0

@scottsykora

I don't think CocoaPods and SwiftPM is ready for this, or I don't think the package manager can supports both Xcode 15 and Xcode 14.

At least SwiftPM does not, when you add .platform(.visionOS(.v1) it will cause compile error on Xcode 14.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

visionOS Apple Vision

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants