On Mon, 17 Feb 2014, Matt W. Benjamin wrote: > Hi Sage, > > I originally used this, and it was replaced with the Boost version as part > of a bugfix. I wonder if the bug was our internal class asserting because of an overlapping interval due to the (start, end) thing? s > > Matt > > ----- "Sage Weil" <sage@xxxxxxxxxxx> wrote: > > > Hi Matt, > > > > I pushed wip-interval-set, which replaces the boost interval_set with > > our > > internal type. As far as I can see it is only being used for > > intersects(), which we support. As long as the interval_set isn't > > supposed to handle overlapping sets (ours doesn't) this should be > > fine. > > > > I did notice one discrepancy where one user was passing in (start, > > length) and the other was using (start, end). I switched it to > > (start, > > offset) (since that's what our class does) but didn't verify the > > semantics > > were what was originally intended... can you take a look? > > > > https://github.com/ceph/ceph/pull/1253 > > > > Thanks! > > sage > > > > > > On Fri, 14 Feb 2014, Matt W. Benjamin wrote: > > > > > yarg, sorry > > > > > > ----- "Josh Durgin" <josh.durgin@xxxxxxxxxxx> wrote: > > > > > > > ICL was added in 1.46 according to > > http://www.boost.org/doc/libs/. > > > > This also fails on debian wheezy with only 1.42 available: > > > > > > > > http://tracker.ceph.com/issues/7422 > > > > > > > > On 02/14/2014 04:26 PM, Matt W. Benjamin wrote: > > > > > according to google, 1.41 ahould have it. > > > > > > > > > > ----- "Sage Weil" <sage@xxxxxxxxxxx> wrote: > > > > > > > > > >> On Fri, 14 Feb 2014, Matt W. Benjamin wrote: > > > > >>> but we haven't built on centos, to my knowledge... > > > > >> > > > > >> I think the question is really: is there an interval_set in > > boost > > > > 1.41 > > > > >> > > > > >> that you can use, can you use something else instead, or do we > > need > > > > to > > > > >> do > > > > >> a boost backport in order to build for centos/rhel 6.x? > > > > >> > > > > >> sage > > > > >> > > > > >>> > > > > >>> ----- "Matt W. Benjamin" <matt@xxxxxxxxxxxx> wrote: > > > > >>> > > > > >>>> oh, ok. I think they need an extra boost rpm? > > > > >>>> > > > > >>>> ----- "Sage Weil" <sage@xxxxxxxxxxx> wrote: > > > > >>>> > > > > >>>>> Hey Matt, > > > > >>>>> > > > > >>>>> This is in the libcephfs code I merged yesterday. Any > > ideas? > > > > >>>>> > > > > >>>>> Thanks! > > > > >>>>> sage > > > > >>>>> > > > > >>>>> > > > > >>>>> On Fri, 14 Feb 2014, Sandon Van Ness wrote: > > > > >>>>> > > > > >>>>>> > > > > >>>>>> The last couple builds of master have been failing on the > > > > >> centos > > > > >>>>> gitbuilder. > > > > >>>>>> here is the build log: > > > > >>>>>> > > > > >>>>>> > > > > >>>>> > > > > >>>> > > > > >> > > > > > > http://gitbuilder.sepia.ceph.com/gitbuilder-centos6-amd64/log.cgi?log=f47062d8a66b94743888aaea13e0766579379395 > > > > >>>>>> > > > > >>>>>> Its running into: > > > > >>>>>> > > > > >>>>>> CXX mds/Capability.lo > > > > >>>>>> CXX mds/Dumper.lo > > > > >>>>>> In file included from client/Client.h:38, > > > > >>>>>> from libcephfs.cc:21: > > > > >>>>>> > > > > >>>>>> error: client/barrier.h:25:38: > > > > >> boost/icl/interval_set.hpp: > > > > >>>> No > > > > >>>>> such > > > > >>>>>> file or directory > > > > >>>>>> > > > > >>>>>> It looks like its not effecting fedora gitbuilders that > > have > > > > >> a > > > > >>>> newer > > > > >>>>> version > > > > >>>>>> of boost but on centos/rhel the version is only 1.41.0-18. > > Is > > > > >>>> there > > > > >>>>> something > > > > >>>>>> we can do to get around this without back-porting or will > > > > >>>>> backporting a newer > > > > >>>>>> boost for rhel/centos be necessary? > > > > >>>>>> -- > > > > >>>>>> 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 > > > > >>>>>> > > > > >>>>>> > > > > >>>> > > > > >>>> -- > > > > >>>> Matt Benjamin > > > > >>>> The Linux Box > > > > >>>> 206 South Fifth Ave. Suite 150 > > > > >>>> Ann Arbor, MI 48104 > > > > >>>> > > > > >>>> http://linuxbox.com > > > > >>>> > > > > >>>> tel. 734-761-4689 > > > > >>>> fax. 734-769-8938 > > > > >>>> cel. 734-216-5309 > > > > >>>> -- > > > > >>>> 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 > > > > >>> > > > > >>> -- > > > > >>> Matt Benjamin > > > > >>> The Linux Box > > > > >>> 206 South Fifth Ave. Suite 150 > > > > >>> Ann Arbor, MI 48104 > > > > >>> > > > > >>> http://linuxbox.com > > > > >>> > > > > >>> tel. 734-761-4689 > > > > >>> fax. 734-769-8938 > > > > >>> cel. 734-216-5309 > > > > >>> > > > > >>> > > > > > > > > > > > > > -- > > > > 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 > > > > > > -- > > > Matt Benjamin > > > The Linux Box > > > 206 South Fifth Ave. Suite 150 > > > Ann Arbor, MI 48104 > > > > > > http://linuxbox.com > > > > > > tel. 734-761-4689 > > > fax. 734-769-8938 > > > cel. 734-216-5309 > > > > > > > > -- > Matt Benjamin > The Linux Box > 206 South Fifth Ave. Suite 150 > Ann Arbor, MI 48104 > > http://linuxbox.com > > tel. 734-761-4689 > fax. 734-769-8938 > cel. 734-216-5309 > > -- 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
