[MacPorts] #42858: [NEW] suricata 1.4.7
#42858: [NEW] suricata 1.4.7 ------------------------+-------------------------------- Reporter: jul_bsd@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Keywords: | Port: suricata ------------------------+-------------------------------- Open Source IDS / IPS / NSM engine * compile/run * devel subport -- Ticket URL: <https://trac.macports.org/ticket/42858> MacPorts <http://www.macports.org/> Ports system for OS X
#42858: [NEW] suricata 1.4.7 -------------------------+-------------------------------- Reporter: jul_bsd@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: suricata | -------------------------+-------------------------------- Comment (by jul_bsd@…): * port lint --nitpick * livecheck * /tab/spacex4/ -- Ticket URL: <https://trac.macports.org/ticket/42858#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#42858: [NEW] suricata 1.4.7 -------------------------+-------------------------------- Reporter: jul_bsd@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: suricata | -------------------------+-------------------------------- Comment (by cal@…): - `--prefix=${prefix}` is automatically passed configure in `configure.pre_args` by MacPorts. You don't have to pass it again. - `--sysconfdir` defaults to `PREFIX/etc`, so passing `--sysconfdir=${prefix}/etc` isn't necessary. - The same applies to `--localstatedir`. - `revision 0` is the default and can be omitted. - Don't set `configure.compiler` directly unless absolutely necessary. Instead, use `compiler.blacklist` to blacklist the compilers that will fail to compile suricata and let MacPorts choose a fallback. Even better would be fixing the build with clang. - Conflicts should be specified both ways, so the suricata subport should `conflicts ${name}-devel` - Instead of listing a variant that does nothing, consider commenting it. - I'm not sure hardcoding the startupitem to the `en1` interface is a good solution. I don't have a better idea either, though. - When adding optional features with dependencies in variants you need to make sure the configure script doesn't autodetect the presence of the dependency and links against libraries when the dependency isn't specified. For example, suricata must not link against libgeoip when installed without the geoip variant even when the libgeoip port is installed. To do this, pass `--disable-geoip` in the default `configure.args` and replace it with `--enable-geoip` in the geoip variant. -- Ticket URL: <https://trac.macports.org/ticket/42858#comment:2> MacPorts <http://www.macports.org/> Ports system for OS X
#42858: [NEW] suricata 1.4.7 -------------------------+-------------------------------- Reporter: jul_bsd@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: suricata | -------------------------+-------------------------------- Comment (by cal@…): Also, configure picks up python as `$prefix/bin/python` {{{ checking for python... /opt/local/bin/python }}} which is a symlink that can be adjusted by users and might cause the package to build different depending on what a user has chosen as python version. You should choose a version of python you want to use and set `configure.python` to avoid that to make the build reproducible. -- Ticket URL: <https://trac.macports.org/ticket/42858#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#42858: [NEW] suricata 1.4.7 -------------------------+-------------------------------- Reporter: jul_bsd@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: suricata | -------------------------+-------------------------------- Comment (by cal@…): Also, here's a patch to fix the build problem with clang due to the system headers defining `strlcat(3)` and `strlcpy(3)` as macros: {{{ #!patch --- configure.ac.orig 2014-04-20 10:01:53.000000000 +0200 +++ configure.ac 2014-04-20 10:02:02.000000000 +0200 @@ -155,7 +155,7 @@ # Checks for library functions. AC_FUNC_MALLOC AC_FUNC_REALLOC - AC_CHECK_FUNCS([gettimeofday memset strcasecmp strchr strdup strerror strncasecmp strtol strtoul memchr]) + AC_CHECK_FUNCS([gettimeofday memset strcasecmp strchr strdup strerror strncasecmp strtol strtoul memchr strlcat strlcpy]) # Add large file support AC_SYS_LARGEFILE --- src/suricata-common.h.orig 2014-04-20 09:52:58.000000000 +0200 +++ src/suricata-common.h 2014-04-20 09:54:45.000000000 +0200 @@ -304,8 +304,16 @@ #include "util-optimize.h" #include "util-path.h" +#ifndef strlcat +/* strlcat is a macro on some systems; if it is, we probably don't need this + * declaration and the macro expansion will break the declaration. */ size_t strlcat(char *, const char *src, size_t siz); +#endif +#ifndef strlcpy +/* strlcpy is a macro on some systems; if it is, we probably don't need this + * declaration and the macro expansion will break the declaration. */ size_t strlcpy(char *dst, const char *src, size_t siz); +#endif #endif /* __SURICATA_COMMON_H__ */ }}} Please submit this patch upstream as well. -- Ticket URL: <https://trac.macports.org/ticket/42858#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#42858: [NEW] suricata 1.4.7 -------------------------+-------------------------------- Reporter: jul_bsd@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: suricata | -------------------------+-------------------------------- Comment (by jul_bsd@…): * update to 2.0 and subport suricata1, disabled suricata-devel for now, review patch for 2.0 * 2.0 is NOT compiling, see main.log * add test command but not sure if it is executed * review livecheck for 2.0 -- Ticket URL: <https://trac.macports.org/ticket/42858#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
#42858: [NEW] suricata 1.4.7 -------------------------+-------------------------------- Reporter: jul_bsd@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: suricata | -------------------------+-------------------------------- Comment (by jul_bsd@…): * suricata 2.0.1 + subport suricata1 => both install/run * configure.blacklist need to be reviewed but seems to be a long one. works ok w gcc4.9 * universal variant in work in progress but failing if enable build.args (be it in universal or not) {{{ util-affinity.h:70:5: error: unknown type name 'cpu_set_t' }}} * test.run * switch tasks to LaunchDaemons * newsyslog configuration to rotate logs * fix livecheck -- Ticket URL: <https://trac.macports.org/ticket/42858#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#42858: [NEW] suricata 1.4.7 -------------------------+-------------------------------- Reporter: jul_bsd@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: suricata | -------------------------+-------------------------------- Comment (by jul_bsd@…): * bump to 2,0,3, -devel 2,1beta1 and keep old 1,4,7 * Merge some patch from duplicate submission #44227 (#44691 "port search: return Trac ticket for submission" would mostly avoid that...) * the 3 versions are up to destroot and 2,0,3+universal * add variant nss -- Ticket URL: <https://trac.macports.org/ticket/42858#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
#42858: [NEW] suricata 1.4.7 -------------------------+-------------------------------- Reporter: jul_bsd@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: suricata | -------------------------+-------------------------------- Comment (by jul_bsd@…): sadly, running test mode 2,1/devel and also 2,0,3 gives {{{ $ sudo suricata -c /opt/local/etc/suricata/suricata.yaml -T 30/8/2014 -- 20:07:45 - <Info> - Running suricata under test mode 30/8/2014 -- 20:07:45 - <Notice> - This is Suricata version 2.1beta1 RELEASE /usr/share/file/magic/animation, 712: Warning: Printf format `l' is not valid for type `belong' in description `%ld x' /usr/share/file/magic/animation, 713: Warning: Printf format `l' is not valid for type `belong' in description `%ld' /usr/share/file/magic/animation, 720: Warning: Printf format `l' is not valid for type `belong' in description `%ld x' /usr/share/file/magic/animation, 721: Warning: Printf format `l' is not valid for type `belong' in description `%ld' /usr/share/file/magic/animation, 785: Warning: Current entry does not yet have a description for adding a MIME type /usr/share/file/magic/animation, 787: Warning: Current entry does not yet have a description for adding a MIME [...] /usr/share/file/magic/xwindows, 34: Warning: Printf format `h' is not valid for type `leshort' in description `version %hd' /usr/share/file/magic/xwindows, 35: Warning: Printf format `h' is not valid for type `leshort' in description `.%hd' 30/8/2014 -- 20:07:45 - <Error> - [ERRCODE: SC_ERR_MAGIC_LOAD(197)] - magic_load failed: File 5.19 supports only version 12 magic files. `/usr/share/file/magic.mgc' is version 7 }}} -- Ticket URL: <https://trac.macports.org/ticket/42858#comment:8> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts