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

[travis] Test that simbody installs. #322

Conversation

chrisdembia
Copy link
Member

No description provided.

@chrisdembia
Copy link
Member Author

@sherm1 if you like this I'll do the same for master.

@scpeters
Copy link
Member

+1

@sherm1
Copy link
Member

sherm1 commented Dec 31, 2014

Great idea, Chris!

@sherm1
Copy link
Member

sherm1 commented Dec 31, 2014

Installed with no errors on Travis (Linux and Mac). Merging. Yes, please do the same for master.

sherm1 added a commit that referenced this pull request Dec 31, 2014
[travis] Test that simbody installs.
@sherm1 sherm1 merged commit 7a4c6a9 into simbody:fix-simbody3.5-Mac-install-problem Dec 31, 2014
@sherm1
Copy link
Member

sherm1 commented Dec 31, 2014

In fact if you make this change for master before we fix the installation script, it will be interesting to see if it fails on OSX. It might depend on the CMake version in use there.

@sherm1
Copy link
Member

sherm1 commented Jan 1, 2015

Both fixes are in the cherry-picks that I used for PR #323, ready for review.

@chrisdembia chrisdembia deleted the travis-test-install branch January 1, 2015 02:08
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

3 participants