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" <[email protected]> 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" <[email protected]> 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" <[email protected]> wrote:
> > >>>
> > >>>> oh, ok. I think they need an extra boost rpm?
> > >>>>
> > >>>> ----- "Sage Weil" <[email protected]> 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 [email protected]
> > >>>>>> 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 [email protected]
> > >>>> 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 [email protected]
> > 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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
