Skip to content

[Do Not Merge] Add OSX to Travis config #1708

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

Closed
wants to merge 2 commits into from
Closed

Conversation

zware
Copy link
Member

@zware zware commented May 21, 2017

Testing feasibility of adding osx back to the Travis config. This should not be merged until it's done a few builds to get an idea of how it affects us.

Just adding a bogus comment to make Travis actually build.
@AraHaan
Copy link
Contributor

AraHaan commented May 22, 2017

Uh the Travis build can take a lot longer than it would with Doozer.io. You guys might need to look into it as on OSX you can tell it to use gcc or clang for building cpython. For more information on setting it up you are free to look at my copy I did to try to bring C++ 14 to the OSX builds which ends up failing completely. It is located at https://github.com/AraHaan/bin2c/blob/master/.doozer.json.

The Doozer.io homepage is at https://doozer.io/ as well.

@refi64
Copy link
Contributor

refi64 commented May 22, 2017

FYI instead of bogus commits, you can close and re-open the PR in order to restart a Travis build.

@AraHaan
Copy link
Contributor

AraHaan commented May 22, 2017

Come to look at it the Linux builds except for the coverage ones could be sped up a lot too with Doozer.io and then only use Travis for coverage.

With that said I will cook up an .doozer.json file for cpython for the normal and doc builds for both OSX and Linux. But on Linux I will not bother apt getting the latest gcc unless cpython recommends a version newer than the default one they ship.

@zware
Copy link
Member Author

zware commented May 22, 2017

@AraHaan This is not the place to discuss replacing Travis. If you're interested in that, either open an issue in python/core-workflow or send a message to the core-workflow list.

@kirbyfan64 Not in this case, since the build was skipped due to not having any code changes.

@zware
Copy link
Member Author

zware commented Jun 3, 2017

Superseded by GH-1846.

@zware zware closed this Jun 3, 2017
@zware zware deleted the travis_osx branch June 3, 2017 17:39
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.

4 participants