On Sat, Mar 14, 2009 at 02:34:31PM -0700, Martin Voros wrote: > > ----- Original Message ---- > > > From: Roman Divacky <rdiva...@freebsd.org> > > To: src-committ...@freebsd.org; svn-src-all@freebsd.org; > > svn-src-h...@freebsd.org > > Sent: Sunday, March 15, 2009 4:55:16 AM > > Subject: svn commit: r189801 - in head: cddl cddl/lib/libzpool > > cddl/usr.bin/ztest cddl/usr.sbin/zdb share/mk > > tools/regression/include/tgmath > > > > Author: rdivacky > > Date: Sat Mar 14 17:55:16 2009 > > New Revision: 189801 > > URL: http://svn.freebsd.org/changeset/base/189801 > > > > Log: > > Switch over to gnu99 compilation on default for userland. > > > > Tested by: make universe > > Tested by: ports exp build (done by pav) > > Reviewed by: ru > > Reviewed by: silence on arch > > Approved by: ed (mentor) > > > > Modified: > > head/cddl/Makefile.inc > > head/cddl/lib/libzpool/Makefile > > head/cddl/usr.bin/ztest/Makefile > > head/cddl/usr.sbin/zdb/Makefile > > head/share/mk/bsd.sys.mk > > head/tools/regression/include/tgmath/Makefile > > > > Modified: head/cddl/Makefile.inc > > ============================================================================== > > --- head/cddl/Makefile.inc Sat Mar 14 17:54:58 2009 (r189800) > > +++ head/cddl/Makefile.inc Sat Mar 14 17:55:16 2009 (r189801) > > @@ -6,3 +6,5 @@ OPENSOLARIS_SYS_DISTDIR= ${.CURDIR}/../. > > IGNORE_PRAGMA= YES > > > > CFLAGS+= -DNEED_SOLARIS_BOOLEAN > > + > > +CSTD?= gnu89 > > Hi, > > I assume that you should have gnu99 here.
no... cddl/* stuff does not build with gnu99 so I use there gnu89 (it's contributed source with -Werror turned on so I wanted to play safe). the subsequent CSTD=c99 are there because some parts needs c99 compilation.
pgpko8t9fGvvr.pgp
Description: PGP signature