Created attachment 201359 [details] Change from libGeoIP to libmaxminddb and bump PORTREVISION ModSecurity 3 supports both libGeoIP and libmaxminddb out of the box, so changing the LIB_DEPENDS is sufficient to make the move.
Looks good to me.
Here's the poudriere buildlog for completeness: https://pkg.cainites.net/data/freebsd_12x64-system/2019-01-23_14h38m39s/logs/modsecurity3-3.0.3_2.log
whit this Update, the i386 architecture stop with a error: http://joneumbox.org/data/112i386-ports/2019-01-28_10h54m10s/logs/errors/modsecurity3-3.0.3_2.log http://joneumbox.org/data/120i386-ports/2019-01-28_11h05m37s/logs/errors/modsecurity3-3.0.3_2.log http://joneumbox.org/data/13i386-ports/2019-01-28_11h06m11s/logs/errors/modsecurity3-3.0.3_2.log The amd64 architecture is fine.
I figured out what the problem is. When compiling libmaxminddb on i386 with clang it incorrectly determines unsigned __int128 is available. Using clang: checking for unsigned __int128... no checking for unsigned int __attribute__((mode(TI)))... yes Using gcc: checking for unsigned __int128... no checking for unsigned int __attribute__((mode(TI)))... no This results in an incorrect maxminddb_config.h, which in turn can make dependant software fail. Since I have no idea how to fix libmaxminddb using clang, my solution would be to compile it with GCC on i386. If you agree I'll create a PR for libmaxminddb as well.
PR created for net/libmaxminddb (see Depends on)
A commit references this bug: Author: joneum Date: Wed Feb 6 16:11:06 UTC 2019 New revision: 492312 URL: https://svnweb.freebsd.org/changeset/ports/492312 Log: Change from libGeoIP to libmaxminddb and bump PORTREVISION PR: 235154 Reported by: Ralf van der Enden <tremere@cainites.net> Approved by: Marius Halden <marius.halden@modirum.com> (maintainer) Sponsored by: Netzkommune GmbH Changes: head/security/modsecurity3/Makefile
Landed. Thx :-)