[Bro] Bro 1.3 available for pre-testing (esp. OpenBSD)

[jean-philippe luiggi]

Hello,

I tested the previous testing release on OpenBSD and the compile stage
worked well.

Very good job !!

I've (too) some remarks :

1°) There was a little problem with "bro.rc" which didn't want to run
because we use a function called "stop()", i suggest to call it
"brostop()" in order to evade some problems with the shell :-)

2°) I worked on the (in)famous "DNS non blocking" section,
if we add the "libbind" package (using 'pkg_add libbind'), we should
have the non blocking features. 
So i tried but i think there's a typo's problem inside "configure.in"

+-+-+
dnl ################################################
dnl # DNS resolver checks.
dnl ################################################
dnl
dnl Check whether our arpa/nameser.h provides type ns_msg.
dnl If not, we disable nonblocking DNS lookups.
dnl We assume worst case first and improve on it below.
AM_CONDITIONAL(USE_NBDNS, false)

dnl Add potential header locations to path
if test -d /usr/local/bind/include; then
   CFLAGS="$CFLAGS -L/usr/local/bind/include"
fi
+-+-+

The last test (with CFLAGS's definition) could not work with OpenBSD
because we should find the include's file in 
"/usr/local/include/bind" (if we added the libbind's package).

I changed (added) the following :
+-+-+
if test -d /usr/local/include/bind; then
   CFLAGS="$CFLAGS -I/usr/local/include/bind"
fi
+-+-+
and now the configure's test for DNS works fine.


I discovered another problem with libmagic (the problem i run into is
relevant to "magic.h" and OpenBSD).
We should specify in the Makefile the correct location for the include
file so i'm now trying to use (in configure.in)  :

+-+-+
        openbsd*)
        AM_CONDITIONAL(USE_NMALLOC, true)
        AC_DEFINE(HAVE_OPENBSD,,[We are on a OpenBSD system])
        LDFLAGS="${LDFLAGS} -L/usr/local/lib"
        V_INCLS="${V_INCLS} -I/usr/local/include"
        CXXFLAGS="${CXXFLAGS} -I/usr/local/include"
        ;;
+-+-+

It's may be a correct workaround (speaking of OpenBSD) as all packages
are put in /usr/local/{include/lib}


Best regards,

And again, very very good job.


Jean-philippe.


On Fri, 06 Jul 2007 17:55:59 -0700
Christian Kreibich <christian at whoop.org> wrote:

> Fyi, the latest 1.3 pre-release is now available at:
> 
>   http://www.icir.org/christian/bro/bro-1.2.19.tar.gz
> 
> This one has additional gcc 4.x compatibility fixes and should also no
> longer cause the regex parser errors some people saw earlier:
> http://www.bro-ids.org/wiki/index.php/"Error_compiling_pattern"
> 
> Cheers,
> Christian
> -- 
> ________________________________________________________________________
>                                            http://www.icir.org/christian
>                                                     http://www.whoop.org
> 
> _______________________________________________
> Bro mailing list
> bro at bro-ids.org
> http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/bro
> 
> 
> 
> 
> 
> !DSPAM:1,468ee813282231804284693!