Skip to content

Commit ed48ad3

Browse files
authored
Merge pull request #488 from ReactiveCocoa/cl-linux-fixes
Correct some Linux-specific build issues
2 parents 2cb86e8 + d7eb8bc commit ed48ad3

File tree

3 files changed

+22
-4
lines changed

3 files changed

+22
-4
lines changed

.travis.yml

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,8 @@ jobs:
8484
env:
8585
- JOB=SWIFTPM_DARWIN
8686
- osx_image: xcode9
87+
before_install:
88+
- eval "$(curl -sL https://gist.githubusercontent.com/kylef/5c0475ff02b7c7671d2a/raw/9f442512a46d7a2af7b850d65a7e9bd31edfb09b/swiftenv-install.sh)"
8789
script:
8890
- swift --version
8991
- swift build
@@ -92,6 +94,7 @@ jobs:
9294
submodules: false
9395
env:
9496
- JOB=SWIFT_4
97+
- SWIFT_VERSION=DEVELOPMENT-SNAPSHOT-2017-07-13-a
9598
- os: linux
9699
language: generic
97100
sudo: required
@@ -105,6 +108,21 @@ jobs:
105108
git:
106109
submodules: false
107110
env: JOB=SWIFTPM_LINUX
111+
- os: linux
112+
language: generic
113+
sudo: required
114+
dist: trusty
115+
before_install:
116+
- eval "$(curl -sL https://gist.githubusercontent.com/kylef/5c0475ff02b7c7671d2a/raw/9f442512a46d7a2af7b850d65a7e9bd31edfb09b/swiftenv-install.sh)"
117+
script:
118+
- swift --version
119+
- swift build
120+
- SWIFTPM_TEST_ReactiveSwift=YES swift test
121+
git:
122+
submodules: false
123+
env:
124+
- JOB=SWIFTPM_SWIFT_4_LINUX
125+
- SWIFT_VERSION=DEVELOPMENT-SNAPSHOT-2017-07-13-a
108126
- stage: prepare carthage cache
109127
script: carthage build --cache-builds
110128
- stage: carthage

Sources/FoundationExtensions.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ extension Date {
8787

8888
extension DispatchTimeInterval {
8989
internal var timeInterval: TimeInterval {
90-
#if swift(>=3.2)
90+
#if swift(>=3.2) && !os(Linux)
9191
switch self {
9292
case let .seconds(s):
9393
return TimeInterval(s)
@@ -117,7 +117,7 @@ extension DispatchTimeInterval {
117117
// This was added purely so that our test scheduler to "go backwards" in
118118
// time. See `TestScheduler.rewind(by interval: DispatchTimeInterval)`.
119119
internal static prefix func -(lhs: DispatchTimeInterval) -> DispatchTimeInterval {
120-
#if swift(>=3.2)
120+
#if swift(>=3.2) && !os(Linux)
121121
switch lhs {
122122
case let .seconds(s):
123123
return .seconds(-s)

Tests/ReactiveSwiftTests/FoundationExtensionsSpec.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ class FoundationExtensionsSpec: QuickSpec {
103103

104104
expect(DispatchTimeInterval.milliseconds(500).timeInterval).to(beCloseTo(0.5))
105105
expect(DispatchTimeInterval.milliseconds(250).timeInterval).to(beCloseTo(0.25))
106-
#if swift(>=3.2)
106+
#if swift(>=3.2) && !os(Linux)
107107
expect(DispatchTimeInterval.never.timeInterval) == Double.infinity
108108
#endif
109109
}
@@ -113,7 +113,7 @@ class FoundationExtensionsSpec: QuickSpec {
113113
expect((-DispatchTimeInterval.milliseconds(1)).timeInterval).to(beCloseTo(-0.001))
114114
expect((-DispatchTimeInterval.microseconds(1)).timeInterval).to(beCloseTo(-0.000001, within: 0.0000001))
115115
expect((-DispatchTimeInterval.nanoseconds(1)).timeInterval).to(beCloseTo(-0.000000001, within: 0.0000000001))
116-
#if swift(>=3.2)
116+
#if swift(>=3.2) && !os(Linux)
117117
expect((-DispatchTimeInterval.never).timeInterval) == Double.infinity
118118
#endif
119119
}

0 commit comments

Comments
 (0)