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

Add support for Xcode 8 and Swift 2.3 #150

Merged
merged 9 commits into from
Aug 18, 2016
Merged

Add support for Xcode 8 and Swift 2.3 #150

merged 9 commits into from
Aug 18, 2016

Conversation

sharplet
Copy link
Contributor

No description provided.

);
runOnlyForDeploymentPostprocessing = 0;
};
D8715DE81C21172A005F4191 /* (null) */ = {

Choose a reason for hiding this comment

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

LOL Xcode

@NachoSoto
Copy link

👍
Can you update .travis.yml too?

github "antitypical/Result" "2.1.1"
github "ReactiveCocoa/ReactiveCocoa" "v4.2.1"
github "antitypical/Result" "2.1.3"
github "ReactiveCocoa/ReactiveCocoa" "v4.2.2"
Copy link
Member

Choose a reason for hiding this comment

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

Explicitly requiring ~> 4.2.2 in Cartfile would be better I think.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, done.

@ikesyo
Copy link
Member

ikesyo commented Jul 25, 2016

pod lib lint in .travis.yml would fail. Maybe we should add --allow-warnings --quick, thus the linter would not validate Xcode project integration.

@ikesyo
Copy link
Member

ikesyo commented Jul 25, 2016

[.travis.yml] xcodebuild build-for-testing test-without-building might be preferable to xcodebuild test on Xcode 8 (at least on Travis) as of Quick/Quick@ed9c17b.

@sharplet
Copy link
Contributor Author

Ok, so it looks like CI passed on Xcode 8 but then failed with this:

/Users/travis/build.sh: line 109: shell_session_update: command not found

Seems like a Travis thing?

And the Xcode 7.3 build just never completes, timing out when building dependencies. Perhaps there should be a separate job for each Xcode/platform combination? That would be 8 total builds, but we'd gain concurrency and more granular failures across platforms.

@sharplet
Copy link
Contributor Author

If someone's available, could https://travis-ci.org/RACCommunity/Rex/builds/151239865 be killed? It was misconfigured and none of the builds are starting (I've since pushed a fix).

@ikesyo
Copy link
Member

ikesyo commented Aug 10, 2016

I've just killed it 🔪

@sharplet
Copy link
Contributor Author

Ok, so the build finally finished. The failures seem to be mostly be timeouts of some kind or another. I'm not sure what the best way to proceed is, the build times here are kinda ridiculous.

@andersio
Copy link
Member

andersio commented Aug 10, 2016

You may try xcodebuild build-for-testing test-without-building with iOS/tvOS, as mentioned by @ikesyo.

#150 (comment)
https://github.com/ReactiveCocoa/ReactiveCocoa/blob/master/.travis.yml

@c0diq
Copy link

c0diq commented Aug 11, 2016

Is there a plan to fix and merge this PR? I am currently blocked because I need this Swift 2.3 support for RAC 4.2.2 Thanks!!

The new build matrix should have 10 items: 2 Xcodes * 5 platforms (Mac,
iOS, tvOS, watchOS, CocoaPods).
@sharplet
Copy link
Contributor Author

Ok, we're green! @c0diq hopefully we can get this in now.

@mdiep
Copy link
Contributor

mdiep commented Aug 18, 2016

Looks good to me. 👍🏻

@mdiep mdiep merged commit 6243fc5 into RACCommunity:master Aug 18, 2016
@sharplet sharplet deleted the swift-2.3 branch August 18, 2016 13:35
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.

6 participants