[Bro-Dev] Fwd: [REL - 10amd64-default][security/bro] Failed for bro-2.2 in build
Sounds like FreeBSD has upgraded to a newer version of clang and it doesn't like to build /src/logging/writers/SQLite.cc anymore. It's the last few lines of the attached message that shows the problem, is there an obvious fix? Thanks! Craig --- Begin Message --- You are receiving this mail as a port that you maintain is failing to build on the FreeBSD package build server. Please investigate the failure and submit a PR to fix build. Maintainer: le...@ee.lbl.gov Last committer: jada...@freebsd.org Ident: $FreeBSD: head/security/bro/Makefile 334597 2013-11-22 15:02:56Z jadawin $ Log URL: http://beefy2.isc.freebsd.org/bulk/10amd64-default/2013-11-29_16h08m05s/logs/bro-2.2.log Build URL: http://beefy2.isc.freebsd.org/bulk/10amd64-default/2013-11-29_16h08m05s Log: >> Building security/bro build started at Sat Nov 30 07:55:51 UTC 2013 port directory: /usr/ports/security/bro building for: FreeBSD 10amd64-default-job-21 10.0-BETA1 FreeBSD 10.0-BETA1 r256420 amd64 maintained by: le...@ee.lbl.gov Makefile ident: $FreeBSD: head/security/bro/Makefile 334597 2013-11-22 15:02:56Z jadawin $ Poudriere version: 3.1-pre ---Begin Environment--- OSVERSION=1000500 UNAME_v=FreeBSD 10.0-BETA1 r256420 UNAME_r=10.0-BETA1 BLOCKSIZE=K MAIL=/var/mail/root PATH=/sbin:/bin:/usr/sbin:/usr/bin:/usr/games:/usr/local/sbin:/usr/local/bin:/root/bin STATUS=1 MASTERMNT=/usr/local/poudriere/data/build/10amd64-default/ref PKG_EXT=txz tpid=20843 POUDRIERE_BUILD_TYPE=bulk PKGNG=1 PKGNAME=bro-2.2 PKG_DELETE=/usr/local/sbin/pkg-static delete -y -f PKG_ADD=/usr/local/sbin/pkg-static add PWD=/root MASTERNAME=10amd64-default USER=root HOME=/root POUDRIERE_VERSION=3.1-pre LOCALBASE=/usr/local PACKAGE_BUILDING=yes PKG_VERSION=/poudriere/pkg-static version PKG_BIN=/usr/local/sbin/pkg-static ---End Environment--- ---Begin OPTIONS List--- ===> The following configuration options are available for bro-2.2: BROCCOLI=on: Build support for libbroccoli communications BROCTL=on: Build BroControl support (requires BROCCOLI) DEBUG=off: Compile in debugging mode IPSUMDUMP=on: Enables traffic summaries LBL_CF=on: Unix time to formated time/date filter support LBL_HF=on: Address to hostname filter support PERFTOOLS=off: Use Perftools to improve memory & CPU usage ===> Use 'make config' to modify these settings ---End OPTIONS List--- --CONFIGURE_ARGS-- --End CONFIGURE_ARGS-- --CONFIGURE_ENV-- TMPDIR="/tmp" PYTHON="/usr/local/bin/python2.7" TMPDIR="/tmp" TMPDIR="/tmp" SHELL=/bin/sh CONFIG_SHELL=/bin/sh --End CONFIGURE_ENV-- --MAKE_ENV-- TMPDIR="/tmp" TMPDIR="/tmp" TMPDIR="/tmp" SHELL=/bin/sh NO_LINT=YES PREFIX=/usr/local LOCALBASE=/usr/local LIBDIR="/usr/lib" CC="cc" CFLAGS="-O2 -pipe -fno-strict-aliasing" CPP="cpp" CPPFLAGS="" LDFLAGS="" CXX="c++" CXXFLAGS="-O2 -pipe -fno-strict-aliasing" MANPREFIX="/usr/local" BSD_INSTALL_PROGRAM="install -s -o root -g wheel -m 555" BSD_INSTALL_LIB="install -s -o root -g wheel -m 444" BSD_INSTALL_SCRIPT="install -o root -g wheel -m 555" BSD_INSTALL_DATA="install -o root -g wheel -m 444" BSD_INSTALL_MAN="install -o root -g wheel -m 444" --End MAKE_ENV-- --SUB_LIST-- PREFIX=/usr/local LOCALBASE=/usr/local DATADIR=/usr/local/share/bro DOCSDIR=/usr/local/share/doc/bro EXAMPLESDIR=/usr/local/share/examples/bro WWWDIR=/usr/local/www/bro ETCDIR=/usr/local/etc/bro --End SUB_LIST-- ---Begin make.conf--- USE_PACKAGE_DEPENDS=yes BATCH=yes WRKDIRPREFIX=/wrkdirs PORTSDIR=/usr/ports PACKAGES=/packages DISTDIR=/distfiles /usr/local/etc/poudriere.d/make.conf WITH_PKGNG=yes NO_RESTRICTED=yes DISABLE_MAKE_JOBS=poudriere ---End make.conf--- ===> Cleaning for bro-2.2 === === === ===> bro-2.2 depends on file: /usr/local/sbin/pkg - not found ===>Verifying install for /usr/local/sbin/pkg in /usr/ports/ports-mgmt/pkg ===> Installing existing package /packages/All/pkg-1.2.1.txz Installing pkg-1.2.1... done If you are upgrading from the old package format, first run: # pkg2ng ===> Returning to build of bro-2.2 === === === === ===> License BSD accepted by the user ===> Fetching all distfiles required by bro-2.2 for building === === ===> License BSD accepted by the user ===> Fetching all distfiles required by bro-2.2 for building => SHA256 Checksum OK for bro-2.2.tar.gz. === =
Re: [Bro-Dev] Fwd: [REL - 10amd64-default][security/bro] Failed for bro-2.2 in build
Which clang version is this? I've tried it with a recent version of the clang 3.4 release branch, and that works fine for me. But based on the error message, I'm attaching a patch; does that help by any chance? Robin -- Robin Sommer * Phone +1 (510) 722-6541 * ro...@icir.org ICSI/LBNL* Fax +1 (510) 666-2956 * www.icir.org/robin diff --git a/src/logging/writers/SQLite.cc b/src/logging/writers/SQLite.cc index 46d1f17..25f5cb0 100644 --- a/src/logging/writers/SQLite.cc +++ b/src/logging/writers/SQLite.cc @@ -126,7 +126,7 @@ bool SQLite::DoInit(const WriterInfo& info, int arg_num_fields, fullpath.append(".sqlite"); string tablename; - map::const_iterator it = info.config.find("tablename"); + WriterInfo::config_map::const_iterator it = info.config.find("tablename"); if ( it == info.config.end() ) { MsgThread::Info(Fmt("tablename configuration option not found. Defaulting to path %s", info.path)); ___ bro-dev mailing list bro-dev@bro.org http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev
Re: [Bro-Dev] Fwd: [REL - 10amd64-default][security/bro] Failed for bro-2.2 in build
On 12/03/13 08:09, Robin Sommer wrote: > Which clang version is this? I've tried it with a recent version of > the clang 3.4 release branch, and that works fine for me. I dunno, I've asked them them to include the clang version before; I'll ask again. > But based on the error message, I'm attaching a patch; does that help > by any chance? I'll see if I can get someone to test it. Thanks! Craig ___ bro-dev mailing list bro-dev@bro.org http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev
Re: [Bro-Dev] Fwd: [REL - 10amd64-default][security/bro] Failed for bro-2.2 in build
On 12/03/13 08:09, Robin Sommer wrote: > Which clang version is this? I've tried it with a recent version of > the clang 3.4 release branch, and that works fine for me. > > But based on the error message, I'm attaching a patch; does that help > by any chance? I submitted a PR with your patch and it worked (thanks!). Here's some feedback: > - The SQLite.cc change also needs to be applied to >src/input/readers/Raw.cc and src/input/readers/SQLite.cc. Please >upstream that as you see fit. > - You didn't manage to reproduce the build failures locally because >it's an issue caused by libc++, not clang. If you build with clang >but still use libstdc++, everything works as expected. Craig ___ bro-dev mailing list bro-dev@bro.org http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev