Actually, it's already merged into the master branch [1]. Thanks. [1] https://github.com/ceph/ceph/pull/12565 On Mon, Dec 19, 2016 at 10:43 AM, Willem Jan Withagen <wjw@xxxxxxxxxxx> wrote: > On 19-12-2016 08:02, Mykola Golub wrote: >> This is due to the recently added config option validators [1] (aimed >> to solve the API issue with rbd_default_features option). >> >> It uses std::is_constructible to check if a type exists or just >> forward declared. Apparently, it does not work for clang. Using >> std::is_destructible instead looks like work [2]. >> >> [1] https://github.com/ceph/ceph/pull/12486 >> [2] https://github.com/trociny/ceph/commit/bd584fe797e30d666800cd4d9b440d4b51d1d27e > > Hi Mykola, > > This seems to be a nice workaround. > I'll submit a PR. > > --WjW > > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Jason -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
