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.

Sign up for GitHub

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

Jump to bottom

in travis, build for osx also #250

Merged
merged 1 commit into from
Jun 25, 2018
Merged

in travis, build for osx also #250

merged 1 commit into from
Jun 25, 2018

Conversation

cdunn2001
Copy link
Contributor

test via pull-request

@cdunn2001
Copy link
Contributor Author

I sent an e-mail to Travis support requesting that they turn on multi-os builds for our repo.

@cdunn2001 cdunn2001 force-pushed the travis branch 2 times, most recently from bbb5935 to 4c6aecf Compare April 24, 2015 14:24
cdunn2001 added a commit to cdunn2001/jsoncpp that referenced this pull request Apr 27, 2015
@cdunn2001
run valgrind in Travis CI
88184d1 
Because this runs apt-get, it will not work as-is for OSX. So when
we have OSX in Travis, we will have to wrap this somehow. See open-source-parsers#250.

Closes open-source-parsers#222.
@cdunn2001 cdunn2001 mentioned this pull request Apr 27, 2015
Merged
@cdunn2001
Copy link
Contributor Author 

Hey Christopher, 

As our OS X capacity has improved, I've enabled multi-os support for your repository open-source-parsers/jsoncpp.

Please note that this feature is currently in beta, so do let us know if there is anything you feel can be improved or anything you miss.

Here's what you need to add in your .travis.yml: http://docs.travis-ci.com/user/multi-os/#Setting-.travis.yml

Let us know if there's anything else we can help with, and thanks for your patience.

Happy testing!

Carla

@cdunn2001 cdunn2001 self-assigned this Aug 9, 2015
@cdunn2001 cdunn2001 force-pushed the travis branch 2 times, most recently from 218320f to 84f679f Compare August 10, 2015 00:28
@cdunn2001 cdunn2001 force-pushed the travis branch 3 times, most recently from 55c355b to 7454232 Compare June 25, 2018 01:34
@cdunn2001
In travis-ci, build for osx also
da49859 
Drop gcc b/c it takes too long to install via addon.

Build only static/release, to save VMs. (No shared to debug.)
@cdunn2001 cdunn2001 added the build or testing cmake, meson, continuous integration, or testing related label Jun 25, 2018
@cdunn2001
Copy link
Contributor Author

I've enabled OSX. For Linux, I've dropped gcc because it took so long to install via the "addon". So we now run Clang++ on both Linux and OSX.

I've also dropped the "shared" and "debug" builds for now. If we add them back, we would have 8 jobs; for now, we have only 2, each taking about 1min.

Appveyor is still pretty slow though.

@cdunn2001 cdunn2001 merged commit 80bc776 into open-source-parsers:master Jun 25, 2018
@cdunn2001 cdunn2001 deleted the travis branch June 25, 2018 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@cdunn2001 cdunn2001

Labels
build or testing cmake, meson, continuous integration, or testing related
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cdunn2001