On 19-12-2016 16:45, Jason Dillaman wrote: > Actually, it's already merged into the master branch [1]. Thanks. > > [1] https://github.com/ceph/ceph/pull/12565 Yup, I saw... Thanx for the speedy patch. in the mean time also submitted: Clang/str_list.h: fix clang warning about std::move #12570 So I guess we're working out the odd bits and pieces that Clang brings with it. --WjW > 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 > > > -- 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
