Re: [Bro-Dev] 2.6-beta build error
On Fri, Sep 21, 2018 at 12:00 PM Azoff, Justin S wrote: > I think I remember the older geoip bifs logging this as well, but only once > at startup, not once per call to lookup_location. Yes, looks like it used to be that way. Changed via [1] now in master. - Jon [1] https://github.com/bro/bro/commit/2ede95422bab0c4853b8db7cc15fe5c6e77aa75a ___ bro-dev mailing list bro-dev@bro.org http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev
Re: [Bro-Dev] 2.6-beta build error
> On Sep 21, 2018, at 11:24 AM, Jon Siwek wrote:
>
> On Thu, Sep 20, 2018 at 2:14 PM Michael Dopheide wrote:
>>
>> bro.bif:3630:47: error: cannot convert ‘stat’ to ‘__dev_t {aka long unsigned
>> int}’ in initialization
>
> Thanks, it's fixed by a simple patch [1] now in master.
>
> - Jon
>
> [1] https://github.com/bro/bro/commit/d7097635f4b1ec9adca548b81ee42968629987a5
That fixed the build, but I think I found a new issue.
If bro is built with maxminddb support but the db files are not present yet,
bro logs this twice for each connection:
1537548892.975875 Reporter::ERROR Failed to open GeoIP location database
(lookup_location(Conn::c$id$resp_h))
/srv/bro/spool/installed-scripts-do-not-touch/site/ncsa/./bro-scripts/./conn-add-geodata.bro,
line 23
1537548892.978846 Reporter::ERROR Failed to open GeoIP location database
(lookup_location(Conn::c$id$orig_h))
/srv/bro/spool/installed-scripts-do-not-touch/site/ncsa/./bro-scripts/./conn-add-geodata.bro,
line 19
I think I remember the older geoip bifs logging this as well, but only once at
startup, not once per call to lookup_location.
—
Justin Azoff
___
bro-dev mailing list
bro-dev@bro.org
http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev
Re: [Bro-Dev] 2.6-beta build error
On Thu, Sep 20, 2018 at 2:14 PM Michael Dopheide wrote:
>
> bro.bif:3630:47: error: cannot convert ‘stat’ to ‘__dev_t {aka long unsigned
> int}’ in initialization
Thanks, it's fixed by a simple patch [1] now in master.
- Jon
[1] https://github.com/bro/bro/commit/d7097635f4b1ec9adca548b81ee42968629987a5
___
bro-dev mailing list
bro-dev@bro.org
http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev
Re: [Bro-Dev] 2.6-beta build error
> On Sep 20, 2018, at 3:13 PM, Michael Dopheide wrote:
>
>
> I'm hitting the same error when trying to build release/2.6 or master:
>
> [ 99%] In file included from /usr/local/src/bro/src/Func.cc:702:0:
> bro.bif: In constructor ‘MMDB::MMDB(const char*, stat)’:
> bro.bif:3630:47: error: cannot convert ‘stat’ to ‘__dev_t {aka long unsigned
> int}’ in initialization
>
> CentOS Linux release 7.5.1804
>
> Let me know if you need any other information. As a point of comparison,
> bro-2.5-897 builds fine.
Looks like travis-ci didn't catch this because the travis-job script does not
install libmaxminddb-devel/libmaxminddb-dev
so bro is not built with mmdb support.
—
Justin Azoff
___
bro-dev mailing list
bro-dev@bro.org
http://mailman.icsi.berkeley.edu/mailman/listinfo/bro-dev
