Otherwise approved.

Would it be possible to add NO_ARCH=yes to the port (poudriere suggests it) ?

Hmm, does these messages mean that the port does not run on 10.3?

Perhaps ask for an MFH, since the new version fixes some security bugs according to the changelog?

The patch and logs look fine, but for completeness, do you also have a log with MODSECURITY_DEVEL turned on?

Why did you add the GH_TAGNAME, given that it is the same as the commit for the 0.1.7 release?

In D11733#243242, @ed wrote:

In D11733#243241, @mat wrote:

You should probably add this before the RUN_DEPENDS line:

IGNORE_FreeBSD_10= only for 11+

Well, that would be a bit too aggressive. Most of the tools set up by this port (e.g., Clang) work perfectly fine on 10.x. It's just that some tools on 11.x work better (e.g., readelf).

It still looks a bit fishy to me (as in the Ports Framework needs to be more intelligent) but it works.

Looks OK to me :)

Can you add the following dependencies? (from the poudriere log):

You can move the CONFLICTS line after the LIB_DEPENDS line (with an extra blank line) to squelch the warning about LIB_DEPENDS / CONFLICTS

Adding "PR: 218762" to the commit log is indeed fine.