In D14126#549056, @mandree wrote:

This needs to be updated to the current version, 3.3.1 added bounce processing.

I found https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=225543 and tried building with the 3.3.1 patch found there.

I just read about the pillow PR.

Add version for rrdtool

Adjust RUN_DEPENDS to match setup.py patches

thanks

Don't be so specific with python
change test deps
change do-test

specify 3.6+ in USES= python
add concurrent to USE_PYTHON - port installs /usr/local/bin/mqttwarn
Add OPTIONS based on extras in setup.py

After talking with upstream (https://github.com/jpmens/mqttwarn/issues/441)
I've removed some patches as unrequired.

Updates based on comments

Comment in patch to say why

Suggestions from koobs

I think I've solved this, since there is a pypi solution

Current status

Put ${check_test} in "quotes" when testing

Restore original checks variable definition

Just one loop.

In D25450#565818, @mat wrote:

Also, please don't add a second loop, just do the test in the loop at the end.

• don't rename checks to CHECKS
• keep it all on one line

What is the next step?

Good idea mat.

Make the change generic, so that any check can be IGNOREd

Adding in recommended changes

Add updating entry
create empty log file
Improve pkg-message

Let's use a newer version of the vendor code.

Remove libtool

testing here is good.

FYI, just found out you can run both snort 2 and snort 3 on the same box by using --daq-dir to point to where the snort3 daq installs (/usr/local/lib/daq by default) and it won't pick up the system libs in /usr/local/lib like snort 2 and everything else in the world.

I think we can remove RUN_DEPENDS and add it to LIB_DEPENDS, and add more to LIB_DEPENDS:

In D24263#533978, @bofh wrote:

In D24263#533976, @dvl wrote:

I have a patch from my coworker. I'm about to look at it and I'll report back soon.

Is your coworker's one better than me ? Otherwise I will commit this with quarterly merge.

In D24263#533974, @bofh wrote:

security/snort3: Update version 3.0.0-258=>3.0.0-270

• UNBREAK
• Change compiler for c++14-lang support
• Remove DEBUG and DOCS DESC to use the defaults
• Add CXXFLAGS+=-O1

BUILDISOK:
http://pdr.bofh.network/data/latest-per-pkg/snort3/3.0.0.270,1/

This needs merge-quarterly hence we should commit manually

I have a patch from my coworker. I'm about to look at it and I'll report back soon.

I have not tried that. My coworker was able to reproduce the issue in his VM, without poudriere. He was building from source, not via ports. This is promising.

Initial testing shows this may be related to SSL options. I will know tomorrow.

In D24263#533860, @bofh wrote:

libdaq tested to work. Give me some times to run through my builder and testing it in a live scenario.

Dependencies: https://github.com/snort3/snort3/tree/3.0.0-270#dependencies