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

Fix building for visionOS. #390

Merged
merged 4 commits into from
Feb 6, 2024
Merged

Fix building for visionOS. #390

merged 4 commits into from
Feb 6, 2024

Conversation

Zandor300
Copy link
Member

@Zandor300 Zandor300 commented Feb 6, 2024

Fixes visionOS compilation issues.

Cherry picked from #371

Closes #371

Fixes #370

Thanks @chrisvasselli!

@@ -628,6 +628,9 @@ public enum Device {
case "i386", "x86_64", "arm64": return simulator(mapToDevice(identifier: ProcessInfo().environment["SIMULATOR_MODEL_IDENTIFIER"] ?? "watchOS"))
default: return unknown(identifier)
}
#elseif os(visionOS)
// TODO: Replace with proper implementation for visionOS.
Copy link

@github-actions github-actions bot Feb 6, 2024

Choose a reason for hiding this comment

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

⚠️ TODOs should be resolved (Replace with proper implementa…).
todo Device.generated.swift:632

@@ -1162,6 +1161,9 @@ public enum Device {
return allTVs
#elseif os(watchOS)
return allWatches
#elseif os(visionOS)
// TODO: Replace with proper implementation for visionOS.
Copy link

@github-actions github-actions bot Feb 6, 2024

Choose a reason for hiding this comment

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

⚠️ TODOs should be resolved (Replace with proper implementa…).
todo Device.generated.swift:1165

@@ -1401,6 +1403,9 @@ public enum Device {
}
#elseif os(tvOS)
return nil
#elseif os(visionOS)
// TODO: Replace with proper implementation for visionOS.
Copy link

@github-actions github-actions bot Feb 6, 2024

Choose a reason for hiding this comment

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

⚠️ TODOs should be resolved (Replace with proper implementa…).
todo Device.generated.swift:1407

@@ -1554,6 +1559,9 @@ extension Device: CustomStringConvertible {
case .simulator(let model): return "Simulator (\(model.description))"
case .unknown(let identifier): return identifier
}
#elseif os(visionOS)
// TODO: Replace with proper implementation for visionOS.
Copy link

@github-actions github-actions bot Feb 6, 2024

Choose a reason for hiding this comment

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

⚠️ TODOs should be resolved (Replace with proper implementa…).
todo Device.generated.swift:1563

@@ -1687,6 +1695,9 @@ extension Device: CustomStringConvertible {
case .simulator(let model): return "Simulator (\(model.safeDescription))"
case .unknown(let identifier): return identifier
}
#elseif os(visionOS)
// TODO: Replace with proper implementation for visionOS.
Copy link

@github-actions github-actions bot Feb 6, 2024

Choose a reason for hiding this comment

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

⚠️ TODOs should be resolved (Replace with proper implementa…).
todo Device.generated.swift:1699

@@ -2326,6 +2337,9 @@ extension Device {
case .simulator(let model): return model.cpu
case .unknown: return .unknown
}
#elseif os(visionOS)
// TODO: Replace with proper implementation for visionOS.
Copy link

@github-actions github-actions bot Feb 6, 2024

Choose a reason for hiding this comment

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

⚠️ TODOs should be resolved (Replace with proper implementa…).
todo Device.generated.swift:2341

@@ -2377,6 +2391,9 @@ extension Device.CPU: CustomStringConvertible {
case .s9: return "S9"
case .unknown: return "unknown"
}
#elseif os(visionOS)
// TODO: Replace with proper implementation for visionOS.
Copy link

@github-actions github-actions bot Feb 6, 2024

Choose a reason for hiding this comment

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

⚠️ TODOs should be resolved (Replace with proper implementa…).
todo Device.generated.swift:2395

Copy link

github-actions bot commented Feb 6, 2024

1 Warning
⚠️ Plist changed, don't forget to localize your plist values

SwiftLint found issues

Warnings

File Line Reason
Device.generated.swift 632 TODOs should be resolved (Replace with proper implementa...). (todo)
Device.generated.swift 1165 TODOs should be resolved (Replace with proper implementa...). (todo)
Device.generated.swift 1407 TODOs should be resolved (Replace with proper implementa...). (todo)
Device.generated.swift 1563 TODOs should be resolved (Replace with proper implementa...). (todo)
Device.generated.swift 1699 TODOs should be resolved (Replace with proper implementa...). (todo)
Device.generated.swift 2341 TODOs should be resolved (Replace with proper implementa...). (todo)
Device.generated.swift 2395 TODOs should be resolved (Replace with proper implementa...). (todo)

Generated by 🚫 Danger

@Zandor300 Zandor300 merged commit fe41d18 into master Feb 6, 2024
16 checks passed
furiosFast added a commit to furiosFast/DeviceKit that referenced this pull request May 31, 2024
* master:
  Version bump. (v5.4.0)
  Add unknown orientation variant (devicekit#411)
  Version bump. (v5.3.1)
  Fix path for PrivacyInfo.xcprivacy in test target.
  Process Privacy manifest instead of copy on SPM. (devicekit#409)
  Version bump. (v5.3.0)
  Support for new May 2024 devices. (devicekit#412)
  Fix ruby version not found. (devicekit#414)
  Version bump. (v5.2.4)
  Enable MERGEABLE_LIBRARY. (devicekit#407)
  Update cocoapods to v1.15.2 in Gemfile.
  Release 5.2.3
  fix: Use cocoapod resource_bundles for PrivacyInfo (devicekit#397)
  Fix building for visionOS.  (devicekit#390)
  Fix building SwiftUI previews on macOS. (devicekit#389)
  Release 5.2.1 (devicekit#385)
  Fix PrivacyInfo.xcprivacy not being detected on SPM. (devicekit#384)
  Update swift-tools-version in Package.swift to 5.3 to support Resource files. (devicekit#381)

# Conflicts:
#	Source/Device.swift.gyb
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.

No longer compiles on visionOS
2 participants