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

Add needed Boost libs and OpenSSL headers to CMake config #193

Merged
merged 5 commits into from Jan 24, 2013

Conversation

3noch
Copy link

@3noch 3noch commented Jan 22, 2013

I have only tested this with Visual Studio 2010. Prior to this change, the Visual Studio projects generated by CMake would not build without manual configuration changes to link against the right Boost libraries and include OpenSSL's headers.

@deanberris
Copy link
Member

Have you verified that this works with versions of Boost as far back as 1.51?

@3noch
Copy link
Author

3noch commented Jan 23, 2013

I've only tested against Boost 1.52.

Provide source port number to requests
This reverts commit ac45ccd, reversing
changes made to d51d6e4.
@deanberris
Copy link
Member

Okay, I'm willing to say that 0.9.5 is going to require Boost 1.52. I'm merging this now. LGTM

deanberris added a commit that referenced this pull request Jan 24, 2013
Add needed Boost libs and OpenSSL headers to CMake config
@deanberris deanberris merged commit f3d9bdc into cpp-netlib:0.9-devel Jan 24, 2013
@3noch
Copy link
Author

3noch commented Jan 25, 2013

FYI: I just did some testing against Boost 1.51 and I didn't see any difference in behavior from Boost 1.52. For both of them, all the tests pass (except get_timeout_test which crashes for both versions of Boost) and some of them have detected memory leaks.

@deanberris
Copy link
Member

Thanks Elliot -- can you share more about the get_timeout_test failures in a different issue? I would love to have a fix for this in 0.9.5.

@3noch
Copy link
Author

3noch commented Jan 28, 2013

Sure: #198

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

2 participants