Skip to content

Conversation

rjeczalik
Copy link

@deanberris
Copy link
Member

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?

deanberris added a commit that referenced this pull request Jul 15, 2013
One last small change for CMakeLists and CMAKE_CXX_FLAGS. (#260)
@deanberris deanberris merged commit 60ad281 into cpp-netlib:master Jul 15, 2013
@rjeczalik
Copy link
Author

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.

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.

2 participants