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

Fixed Travis CI for macOS #7962

Merged
merged 3 commits into from
Jun 8, 2018
Merged

Fixed Travis CI for macOS #7962

merged 3 commits into from
Jun 8, 2018

Conversation

Yardanico
Copy link
Collaborator

@Yardanico Yardanico commented Jun 5, 2018

No description provided.

@Yardanico
Copy link
Collaborator Author

@Yardanico
Copy link
Collaborator Author

Yardanico commented Jun 5, 2018

This seems to work, macOS build on travis works now

@Yardanico
Copy link
Collaborator Author

Yardanico commented Jun 5, 2018

Travis failed because I cancelled linux build (to not waste time), and not all tests passed on macOS.
This is ready to be merged if everyone is OK with that solution.

@Yardanico Yardanico changed the title [CI test] Yet another attempt to fix OSX on Travis :/ Fixed Travis CI for macOS Jun 5, 2018
@dom96
Copy link
Contributor

dom96 commented Jun 6, 2018

What does this do? And why is it required for macOS?

Maybe you could look into why the tests fail on macOS too?

@Yardanico
Copy link
Collaborator Author

Yardanico commented Jun 7, 2018

@dom96 This PR fixes building Nim on Travis with macOS (so it can at least run tests now).
I linked two issues which explain why this is required. It happens because Nim travis config uses "set -e", and Travis doesn't seem to like it.

@Yardanico
Copy link
Collaborator Author

@dom96 as you can see in current macOS builds on travis it just fails before Nim compilation: https://travis-ci.org/nim-lang/Nim/jobs/389173091

@dom96 dom96 merged commit caaaa87 into nim-lang:devel Jun 8, 2018
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.

None yet

2 participants