Add a new option, REDIS, to enable redis output. Since this options is OFF
by default, no bump on PORTREVISION was done
Details
Details
Diff Detail
Diff Detail
- Repository
- rP FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
looks good, will accept after typo fix :)
| security/suricata/Makefile | ||
|---|---|---|
| 56 ↗ | (On Diff #29440) | small typo here |
| security/suricata/Makefile | ||
|---|---|---|
| 108 ↗ | (On Diff #29468) | This should probably be a REDIS_CONFIGURE_ENABLE=hiredis so that a --disable-hiredis is added when the option is disabled. See "feature auto-activation" in https://www.freebsd.org/doc/en/books/porters-handbook/makefile-options.html |
| security/suricata/Makefile | ||
|---|---|---|
| 108 ↗ | (On Diff #29468) | the style of the port should be updated, but we can do this after initial merge. I have a larger 3.2.2 update waiting anyway and can look at it then? |
| security/suricata/Makefile | ||
|---|---|---|
| 108 ↗ | (On Diff #29468) | I would rather bad things never go in, there are already enough in this port. |
| security/suricata/Makefile | ||
|---|---|---|
| 108 ↗ | (On Diff #29468) | ok, now I'm seeing this with REDIS disabled, REDIS_CONFIGURE_ENABLE does something that suricata doesn't like. checking hiredis/hiredis.h usability... no checking hiredis/hiredis.h presence... no checking for hiredis/hiredis.h... no ERROR! libhiredis library not found, go get it from https://github.com/redis/hiredis or your distribution: Ubuntu: apt-get install libhiredis-dev Fedora: dnf install hiredis-devel RHEL/CentOS: yum install hiredis-devel ===> Script "configure" failed unexpectedly. Please report the problem to franco@opnsense.org [maintainer] and attach the "/usr/obj/usr/ports/security/suricata/work/suricata-3.2.2/config.log" including the output of the failure of your make command. Also, it might be a good idea to provide an overview of all packages installed on your system (e.g. a /usr/local/sbin/pkg-static info -g -Ea). *** Error code 1 |
| security/suricata/Makefile | ||
|---|---|---|
| 108 ↗ | (On Diff #29468) | What is the content of CONFIGURE_ARGS ? |
| security/suricata/Makefile | ||
|---|---|---|
| 108 ↗ | (On Diff #29468) | $ ./configure --enable-gccprotect --with-libpcre-includes=/usr/local/include --with-libpcre-libraries=/usr/local/lib --with-libyaml-includes=/usr/local/include --with-libyaml-libraries=/usr/local/lib --with-libnet-includes=/usr/local/include --with-libnet-libraries=/usr/local/lib --with-libhtp-includes=/usr/local/include/ --with-libhtp-libraries=/usr/local/lib --disable-gccmarch-native --enable-geoip --enable-bundled-htp --with-libhs-includes=/usr/local/include --with-libhs-libraries=/usr/local/lib --enable-ipfw --with-libjansson-includes=/usr/local/include --with-libjansson-libraries=/usr/local/lib --enable-netmap --with-libpcap-includes=/usr/include --with-libpcap-libraries=/usr/lib --disable-prelude --disable-hiredis --disable-unittests --prefix=/usr/local --localstatedir=/var --mandir=/usr/local/man --disable-silent-rules --infodir=/usr/local/info/ --build=amd64-portbld-freebsd11.0--disable-hiredis seems to achieve the opposite, I verified this manually on 3.2.1 and 3.2.2 |
Comment Actions
So FWIW I stand by the approval of the previous version that does not use REDIS_CONFIGURE_ENABLE.
As a heads-up note: 3.2.2 is out and I will add the update to bugzilla now, commits may collide slightly as PRELUDE dependencies are being adjusted.
| security/suricata/Makefile | ||
|---|---|---|
| 108 ↗ | (On Diff #29468) | Suricata configure scripts only deal with --enable options and don't support --disable-FEATURE that is the cause of confusion. We can report it upstream to see if it can be improved for future versions. I'll update to old patch here to get it committed |
Comment Actions
can you rebase this patch? it looks off with the update to 3.2.2 and no OPTIONS_DEFINE like it was already applied