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

In D22372#513374, @mat wrote:

The *_DEPENDS are out of order, they have to happen earlier.
Also, our style make it having those variable defined at once, in a long continuation, not using +=.

There seems to be a lot of them:

I see this works:

I have a clean testport of the raw diff on FreeBSD 12.1

Introduce do-install-EXAMPLES-on
Move EXAMPLES related command to that stage
Adjust pkg-plist to cover examples directories.

Oh, I commandeer via the web interface.

Commandeer back after help from swills (thank you).

Thank you for the patch file, and for cleaning up the Makefile.

Based on https://netty.io/wiki/forked-tomcat-native.html#wiki-h2-7 we need to invoke: mvn clean install

In D22114#484573, @bofh wrote:

Oops. @dvl sorry. Didn't know that partial commit closes the revision.

In D22114#484562, @bofh wrote:

Hi @dvl ,
Are you sure that snort3 builds after properly with these changes? So far in my system it doesn't build. For your reference please check the following:
http://pdr.bofh.network/data/latest-per-pkg/snort3/3.0.0.262,1/
I am committing libdaq for now. And will look more into it to fix it soon.