Re: sysutils/xsi not building on -CURRENT with new gcc

On Sat, 2007-07-07 at 21:55 +0200, Attilio Rao wrote:
> 2007/7/7, Joe Marcus Clarke <marcus_at_marcuscom.com>:
> > On Sat, 2007-07-07 at 14:32 -0400, Tom McLaughlin wrote:
> > > Hi all,
> > >
> > > I updated sysutils/xsi to a new version and I just found it will not
> > > build on -CURRENT (July 2nd) in my tinderbox.  I'm getting the following
> > > fail:
> > >
> > >
> > > if c++ -DHAVE_CONFIG_H -I. -I. -I. -I../lib -I/usr/local/include/libxml2
> > > -I/usr/local/include -O1 -pipe -MT freebsd.o -MD -MP -MF
> > > ".deps/freebsd.Tpo" -c -o freebsd.o freebsd.cpp;  then mv -f
> > > ".deps/freebsd.Tpo" ".deps/freebsd.Po"; else rm -f ".deps/freebsd.Tpo";
> > > exit 1; fi
> > > In file included from /usr/include/sys/sx.h:37,
> > >                  from /usr/include/sys/filedesc.h:40,
> > >                  from /usr/include/sys/proc.h:44,
> > >                  from /usr/include/sys/user.h:47,
> > >                  from freebsd.cpp:35:
> > > /usr/include/sys/lock_profile.h:156: error: expected ',' or '...' before 'class'
> >
> > Seems like a bug in lock_profile.h to me.  We should not be using
> > reserved words like "class".  kmacy added this function to
> > lock_profile.h in rev 1.6.  You might want to ping him about it.
> 
> Could you please try this:
> http://users.gufi.org/~rookie/works/patches/lock_profile.diff
> 
> Thanks,
> Attilio

Yup, that did it.  xsi just built fine here.

tom

-- 
| tmclaugh at sdf.lonestar.org             tmclaugh at FreeBSD.org |
| FreeBSD                                   http://www.FreeBSD.org |