Re: Upgrade to 2.53.1 and add FLAVOR to the net/i2pd port

On Fri, Aug 30, 2024 at 05:55:20PM +0200, David Uhden Collado wrote:
> > > 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?

ok tb

> 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