- Notifications
You must be signed in to change notification settings - Fork425
One last small change for CMakeLists and CMAKE_CXX_FLAGS. (#260)#274
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
LGTM This looks innocent enough, it should be fine. PS. What is this travis-ci thing, and should we be using this more and configuring it appropriately? |
One last small change for CMakeLists and CMAKE_CXX_FLAGS. (#260)
I wanted to setup continuous integration for testing my local cpp-netlib changes. It is not configured, that's why it's failing. With a tiny bit of work (writing proper .travis.yml configuration) it should be fairly easy to setup CI for cpp-netlib. CI should help with verifying simple bugfixes as build system or for running tests. Unfortunately it's for Linux only, but still it's better this than nothing. If I got spare time I could look into that and propose some cfg if appropriate. [edit] Using Travis for cpp-netlib would probably require having Boost libraries in another git submodule, as worker machines offer Boost 1.48 IIRC. |
1b50308
#260