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

cmake,make-dist: bump up boost version to 1.67 #21572

Merged
merged 1 commit into from Apr 23, 2018

Conversation

tchaikov
Copy link
Contributor

Signed-off-by: Kefu Chai kchai@redhat.com

@tchaikov
Copy link
Contributor Author

it's separated from #21484

* to pick up the fix of https://svn.boost.org/trac10/ticket/11622
* also the boost::python's library name now includes the version suffix
  of python version, so update BuildBoost.cmake accordingly.

Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov self-assigned this Apr 21, 2018
@wjwithagen wjwithagen self-requested a review April 22, 2018 19:35
Copy link
Contributor

@wjwithagen wjwithagen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After upgrading packages to:
boot 1.67
CMake 3.11.0_2
FreeBSD happily compiles

@liewegas liewegas modified the milestone: mimic Apr 23, 2018
@tchaikov tchaikov merged commit f612f0f into ceph:master Apr 23, 2018
@tchaikov tchaikov deleted the wip-boost-1.67 branch April 23, 2018 14:57
@@ -78,7 +78,7 @@ struct bound_completion_handler : public invoker<Result> {
// static check for CompletionHandler concept (must be CopyConstructible and
// callable with no arguments)
using namespace boost::asio;
BOOST_ASIO_COMPLETION_HANDLER_CHECK(bound_completion_handler, *this) type_check;
BOOST_ASIO_LEGACY_COMPLETION_HANDLER_CHECK(bound_completion_handler, *this) type_check;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just leaving a note that this line will go away eventually. it was tagged as 'legacy' because its CopyConstructible requirement was removed. there's work in progress on a generalized async::Completion type in #21271 - once that merges, i plan to use it to replace most of this stuff

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants