Hi Sage, I originally used this, and it was replaced with the Boost version as part of a bugfix. 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
