I prefer to keep UPnP support as a flavor to avoid installing unnecessary
dependencies for users who do not need this functionality.

we don't generally do that in ports, it increases the burden on people
testing an update diff because then they need to check both versions

if it doesn't change the behaviour unless a setting is also changed,
we would usually not add a flavour


Thank you for informing me. I'm new to porting software to OpenBSD and still learning the common practices. If this option doesn't alter the software's behavior, would it be possible to apply Jeremie Courreges-Anglas's diff?
Index: Makefile
===================================================================
RCS file: /home/cvs/ports/net/i2pd/Makefile,v
diff -u -p -r1.25 Makefile
--- Makefile	29 Aug 2024 15:49:13 -0000	1.25
+++ Makefile	29 Aug 2024 21:34:00 -0000
@@ -3,6 +3,7 @@ COMMENT =	client for the I2P anonymous n
 GH_ACCOUNT =	PurpleI2P
 GH_PROJECT =	i2pd
 GH_TAGNAME =	2.53.1
+REVISION =	0
 
 CATEGORIES =	net
 HOMEPAGE =	https://i2pd.website
@@ -12,14 +13,17 @@ PERMIT_PACKAGE = Yes
 
 WANTLIB += ${COMPILER_LIBCXX} boost_atomic-mt boost_date_time-mt
 WANTLIB += boost_filesystem-mt boost_program_options-mt boost_system-mt
-WANTLIB += c crypto m ssl z
+WANTLIB += c crypto m miniupnpc ssl z
 
 COMPILER =	base-clang ports-gcc
 MODULES =	devel/cmake
-LIB_DEPENDS =	devel/boost
+LIB_DEPENDS =	devel/boost \
+		net/miniupnp/miniupnpc
 
 # for tests
 USE_GMAKE =	Yes
+
+CONFIGURE_ARGS =	-DWITH_UPNP=ON
 
 WRKSRC =	${WRKDIST}/build

Reply via email to