Re: Portsentry 2.0 released

[Thomas Klausner <wiz%gatalith.at@localhost>]

Hi Marcus!

On Mon, Jun 23, 2025 at 08:18:29PM +0200, Marcus Hufvudsson wrote:
> Hello! I was advised to send an email to this address from the #pkgsrc IRC
> channel. My name's Marcus and to make a long story short, I have revived the
> long dead Portsentry project and intend to keep maintaining it and implement
> new features. The project recently celebrated the release of version 2.0 and
> I'd like to extend an invitation to package this new version in NetBSD.
> 
> I don't know how to proceed with this or if anyone on your side is
> interested in updating Portsentry (the old package lives here: https://cdn.netbsd.org/pub/pkgsrc/current/pkgsrc/security/portsentry/index.html).
> In my revival and update of the source code I have continually tested on
> NetBSD, the code compiles and run's on the latest, NetBSD 10. The project
> source is hosted on https://github.com/portsentry/portsentry

Thanks for contacting us, and for testing on NetBSD!

I've packaged an upgrade candidate for this in wip/portsentry
(https://pkgsrc.org/wip/).

There's a small patch I'd like to see included in upstream - the
CMakeLists.txt file includes "usr" in an install path, but that leads
to the file being installed into
"/usr/pkg/usr/sbin/portsentry". ${PREFIX} is implied in install rules,
so removing 'usr/' is better.

The package builds and the program starts, but I haven't used it
before, so if you (or someone else who knows it) could do a quick
sanity check that'd be great.

Cheers,
 Thomas

$NetBSD$

${PREFIX} (/usr) is default, don't add it.

--- CMakeLists.txt.orig	2025-06-24 07:38:52.887363491 +0000
+++ CMakeLists.txt
@@ -63,7 +63,7 @@ if (USE_PCAP)
 endif()
 
 # INSTALL TARGETS for portsentry program
-install(TARGETS portsentry DESTINATION usr/sbin)
+install(TARGETS portsentry DESTINATION sbin)
 install(FILES examples/portsentry.conf DESTINATION etc/portsentry)
 install(FILES examples/portsentry.ignore DESTINATION etc/portsentry)