Skip to content

Enable cpp-netlib to be build into a single library #227

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

Merged
merged 4 commits into from
Apr 9, 2013
Merged

Enable cpp-netlib to be build into a single library #227

merged 4 commits into from
Apr 9, 2013

Conversation

torbjoernk
Copy link

Sometimes I find it tedious to include the various cpp-netlib libraries in my project. Thus I added the possibility to build a single library out of all cpp-netlib's separated libraries -- including uri (I'm sending a PR there too).

The configuration of the tests and examples has been adjusted to cope with the fact, that when CPP-NETLIB_BUILD_SINGLE_LIB is enabled, the separated libraries are not build.

Tested with GCC 4.7 and Boost 1.53.

Notes

  1. This PR includes an update for the uri submodule provided by an PR there (see enabling cpp-netlib (and uri) to be compiled into a single library uri#10)
  2. This PR is based on my other PR cpp-netlib-http-client_test fails when triggered through make test #226 fixing http-client_test

when issuing 'make test' the http-client_test were not found due to
wrong parameter in add_test() call
this adds the CMake variable 'CPP-NETLIB_BUILD_SINGLE_LIB', which
defaults to OFF and will enable build of a single library 'libcppnetlib'
to be compiled instead of the different separated libraries.

the tests and examples have been adjusted to deal with a single library

the uri submodule has also been changed
@deanberris
Copy link
Member

LGTM. Thanks for the PR!

deanberris added a commit that referenced this pull request Apr 9, 2013
Enable cpp-netlib to be build into a single library
@deanberris deanberris merged commit 7af8275 into cpp-netlib:master Apr 9, 2013
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