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

Stop testing iOS 7.0.3 on Travis CI #345

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tjarratt
Copy link
Contributor

Adds 8.3 and 8.4 tasks to the Travis Matrix.

Thought that this might be a controversial change and that a pull request would be best.

According to MixPanel, iOS 7.0 accounts for less than ~2% of all iOS users in the last six months.

@akitchen
Copy link
Contributor

I don't think it's that controversial. however we should keep support for the last 7.x release as long as we can, IMO

We're also overdue to specify osx_image

👍

@tjarratt
Copy link
Contributor Author

Totally agree about supporting iOS 7, @akitchen. Seems like one of these changes broke the build horribly, so we'll have to look into that.

I've been quite surprised at how quickly TravisCI has been at updating their osx_image list lately. There's already an Xcode6_4 image available!

@tjarratt
Copy link
Contributor Author

I really wanted to use the osx_image Travis key, but apparently that locks you into a single SDK. When I specified Xcode 6.4 as the osx_image then we only had the ability to build against the 8.4 SDK -- earlier ones were not present.

For more info, check out this issue I opened against Travis-CI.

@briancroom
Copy link
Contributor

This page documents which SDKs are available on the various Travis OS X images: http://docs.travis-ci.com/user/languages/objective-c/

Long story short, the default Xcode 6.1 image is the only one that has more than a single iOS SDK version. This complicates matters as well now that the iOS 9 SDKs are getting added to the mix.

@briancroom briancroom mentioned this pull request Sep 14, 2015
@briancroom briancroom force-pushed the update-travis-ci-matrix branch 3 times, most recently from 1a9ef8e to 1ac61b7 Compare September 23, 2015 19:21
@briancroom
Copy link
Contributor

@tjarratt FYI I've hijacked this PR a bit to include getting Travis to run with Xcode 7/iOS 9 using a newly discovered technique allowing us to use different Travis images for different build variations. Still got a few kinks to work out though!

@tjarratt
Copy link
Contributor Author

tjarratt commented Feb 2, 2016

Rebasing this on top of master. I'd like to get this merged in this week so we can also merge in pivotal-legacy/PivotalCoreKit#168.

@tjarratt tjarratt force-pushed the update-travis-ci-matrix branch 6 times, most recently from d2f0836 to 666f905 Compare February 3, 2016 23:19
- Drop iOS 7 runtime
- Use Xcode 7 for testing the iOS 8.4 and 9.2 runtimes
- Only build against latest iOS 9 SDK available in the Xcode 7.2 image
- Run the UI specs with the Debug configuration so symbolication works properly
- Try to ensure the simulator has run at least once before we run tests. Under Xcode 7
  we need to wait for the asynchronous process of "launching" the simulator.
- Try more aggressively to kill the simulator in-between runs. The simulator process
  was renamed sometime in the Xcode 7 release timeframe, and it may only be called
  "Simulator" now. I anticipate that we may need to do more work in order to wait for
  the simulator to "boot" initially, which was also added in the Xcode 7 release.
I'd love to find a non-flakey solution for test bundles, but I'm just
not finding one at the moment.
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.

3 participants