CEPH filesystem development
 help / color / mirror / Atom feed
From: Casey Bodley <cbodley@redhat.com>
To: The Sacred Order of the Squid Cybernetic <ceph-devel@vger.kernel.org>
Subject: Re: coming in boost 1.66
Date: Tue, 19 Dec 2017 13:56:22 -0500	[thread overview]
Message-ID: <23b03cec-760d-5e32-2232-dfa6b90396f0@redhat.com> (raw)
In-Reply-To: <07a75940-72c6-8510-459b-4f4a0ad43d4b@redhat.com>


On 11/27/2017 11:03 AM, Casey Bodley wrote:
> There are a couple of interesting changes in the upcoming boost 1.66 
> release, based on the work-in-progress release notes: 
> http://www.boost.org/users/history/in_progress.html
>
> - the Beast library was accepted into boost earlier this year, and 
> 1.66 will be its first release. We're currently including it in a 
> submodule, so we'll need to come up with a strategy to transition.
> - the Asio library is getting "interface changes to reflect the 
> Networking TS" (which is awesome!) but we might see some compilation 
> failures in radosgw as a result.
>
> Now that we have the tooling to build specific boost versions, how do 
> people feel about forcing the update to 1.66 once we've got it tested? 
> This could avoid the extra work needed to a) maintain the beast 
> submodule alongside boost's, and b) work around any hypothetical api 
> breakage in asio. Once 1.66 is out, we'll have a better idea how much 
> work that actually requires, but I wanted to float the idea ahead of 
> time.
>
> Casey

Boost 1.66 was released yesterday! The boost::beast changes were minimal 
(only one function name changed), but the changes to boost::asio were 
extensive and necessitated major changes to my PR for librados bindings 
at https://github.com/ceph/ceph/pull/19054. So I'm still proposing that 
we pin our boost dependency at 1.66 to avoid #ifdefs and build hackery 
to continue supporting earlier versions. That PR includes the cmake 
changes to do this, though I'm happy to pull those into a separate PR if 
necessary.

Casey

  parent reply	other threads:[~2017-12-19 18:56 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-27 16:03 coming in boost 1.66 Casey Bodley
2017-11-27 18:19 ` Jesse Williamson
2017-12-19 18:56 ` Casey Bodley [this message]
2017-12-20 20:50   ` Casey Bodley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=23b03cec-760d-5e32-2232-dfa6b90396f0@redhat.com \
    --to=cbodley@redhat.com \
    --cc=ceph-devel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox