Skip to content

Commit 8de70d3

Browse files
committed
Fix optionality issues after revert
1 parent cfb7836 commit 8de70d3

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

Sources/Basics/Vendor/Triple.swift

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1527,7 +1527,7 @@ extension Triple {
15271527
/// `darwin` OS version number is not adjusted to match the equivalent
15281528
/// `macosx` version number. It's usually better to use `version(for:)`
15291529
/// to get Darwin versions.
1530-
public var osVersion: Version {
1530+
public var osVersion: Version? {
15311531
var osName = self.osName[...]
15321532

15331533
// Assume that the OS portion of the triple starts with the canonical name.
@@ -1626,7 +1626,7 @@ extension Triple {
16261626
///
16271627
/// This accessor is semi-private; it's typically better to use `version(for:)` or
16281628
/// `Triple.FeatureAvailability`.
1629-
public var _iOSVersion: Version {
1629+
public var _iOSVersion: Version? {
16301630
switch os {
16311631
case .darwin, .macosx:
16321632
// Ignore the version from the triple. This is only handled because the
@@ -1653,7 +1653,7 @@ extension Triple {
16531653
///
16541654
/// This accessor is semi-private; it's typically better to use `version(for:)` or
16551655
/// `Triple.FeatureAvailability`.
1656-
public var _watchOSVersion: Version {
1656+
public var _watchOSVersion: Version? {
16571657
switch os {
16581658
case .darwin, .macosx:
16591659
// Ignore the version from the triple. This is only handled because the
@@ -1662,7 +1662,7 @@ extension Triple {
16621662
// OS X.
16631663
return Version(2, 0, 0)
16641664
case .watchos:
1665-
var version = self.osVersion
1665+
guard var version = self.osVersion else { return nil }
16661666
if version.major == 0 {
16671667
version.major = 2
16681668
}

Tests/BasicsTests/TripleTests.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -169,14 +169,14 @@ final class TripleTests: XCTestCase {
169169

170170
let macos = try? Triple("x86_64-apple-macosx10.15")
171171
XCTAssertNotNil(macos!)
172-
XCTAssertEqual(macos!.osVersion, .init(parse: "10.15"))
172+
XCTAssertEqual(macos!.osVersion, .init(parse: "10.15")!)
173173
let newVersion = "10.12"
174174
let tripleString = macos!.tripleString(forPlatformVersion: newVersion)
175175
XCTAssertEqual(tripleString, "x86_64-apple-macosx10.12")
176176
let macosNoX = try? Triple("x86_64-apple-macos12.2")
177177
XCTAssertNotNil(macosNoX!)
178178
XCTAssertEqual(macosNoX!.os, .macosx)
179-
XCTAssertEqual(macosNoX!.osVersion, .init(parse: "12.2"))
179+
XCTAssertEqual(macosNoX!.osVersion, .init(parse: "12.2")!)
180180

181181
let android = try? Triple("aarch64-unknown-linux-android24")
182182
XCTAssertNotNil(android)

0 commit comments

Comments
 (0)