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

Fixing nightly build. #1489

Merged
merged 7 commits into from
Mar 26, 2020
Merged

Fixing nightly build. #1489

merged 7 commits into from
Mar 26, 2020

Conversation

noobs2ninjas
Copy link
Member

  • Updated xctoolchain checkout in gitmodules
  • Removed unnecessary code that creates duplicate tvOS simulators

@noobs2ninjas
Copy link
Member Author

noobs2ninjas commented Mar 17, 2020

@drdaz or @TomWFox If you want to verify. I created this branch and then copied the nightly build code to run on commit. You can see the successful test here. Now that its passed I have returned the project back to normal only building PR and commits to master.

@TomWFox
Copy link
Contributor

TomWFox commented Mar 17, 2020

Looks like there is something funny going on with the iOS build now.

@noobs2ninjas noobs2ninjas linked an issue Mar 18, 2020 that may be closed by this pull request
@noobs2ninjas
Copy link
Member Author

noobs2ninjas commented Mar 18, 2020

Not sure what happened. Was a bundler error or something. I just reran it and it seems to have built just fine.

@drdaz
Copy link
Member

drdaz commented Mar 26, 2020

Sorry this got away from me. I've been distracted by, amongst other things, the entire world stopping.

I came to try and fix the nightly, but found this.

I feel like it's a problem that what we test on our PRs isn't quite the same as what we do against master. Is there any reason why we don't run the same targets as the nightlies on our PRs before we merge? I feel like it creates surprises.

Anybody know how to add it to the PR tests?

@drdaz
Copy link
Member

drdaz commented Mar 26, 2020

That comment sounded all negative. This looks like great work 👍🏼

@drdaz drdaz merged commit 4ba0593 into master Mar 26, 2020
@drdaz
Copy link
Member

drdaz commented Mar 26, 2020

@noobs2ninjas Is there some kind of cat and mouse game going on with the Simulator versions and our config here? As in, do we need to keep updating them?

I ask since I think I remember seeing an option to predefine which Simulators were available on the machine. We could get stability that way.

@noobs2ninjas
Copy link
Member Author

@drdaz

Theres been a lot of issues going on between Travis and Circle.
One issue was Travis changed the default version of iOS changed on the same environment with no warning. So where Xcode 11 had 12.0 all the sudden it changed to 12.1.2 and began failing. When we updated that all the sudden we had to update some of our bundle versions for it to be compatible with that environments new ruby version which then new gems deprecated certain verbiage which we had to update. Then we had to manually add OCMock to our project for the builds which updated the projects Xcode version again which then caused another environment update. Then all the sudden for some reason Travis began adding its own tvOS simulator by default making our script incompatible causing yet another update to our scripts. Also with each environment update changed the name of the simulators. The release of the iPhone 11 changed which simulators where available by default. Just one thing after another.

I'm hoping this is the end of it. I'll admit that some of the early adjustments have been due to just me learning. Recently though its like conducting an orchestra with no experience and the time signature keeps changing every 4 measures. Music analogy...check.

I plan on leaving this setup on there until we are absolutely forced to update again. The only update I see happening is when we update the pod spec with the new bolts version once that is released.

@TomWFox TomWFox deleted the Build-Fixes branch July 14, 2020 22:40
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