Re: Master needs newer version of boost on centos?

["Matt W. Benjamin" <matt@linuxbox.com>]

Hi,

I don't know, that's an interesting catch.  I guess it's not really feasible to do
anything accept ack this, for now.

Matt

----- "Sage Weil" <sage@inktank.com> wrote:

> 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@inktank.com> 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@inktank.com> 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@inktank.com> 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@linuxbox.com> wrote:
> > > > > >>>
> > > > > >>>> oh, ok.  I think they need an extra boost rpm?
> > > > > >>>>
> > > > > >>>> ----- "Sage Weil" <sage@inktank.com> 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@vger.kernel.org
> > > > > >>>>>> 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@vger.kernel.org
> > > > > >>>> 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@vger.kernel.org
> > > > > 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 
> > 
> >

-- 
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