On Wed May 21, 2025 at 05:28:31AM +0000, Klemens Nanni wrote:
> net/tdesktop needs net/tdlib with -TD_E2E_ONLY=ON, which is really a second
> port as it exposes private API;  it is a different library in the same repo.
> 
> FLAVORS / MULTI_PACKAGE is not an option due to conflicts and overall complexity.
> 
> Splitting net/tdlib/tdlib and net/tdlib/tde2e with common Makefile.inc makes
> stuff easy, doesn't change anything for existing tdlib consumers (net/py-telegram).
> 
> The diff adjusts the only consumer, attached is net/tdlib/ without CVS bits.
> 
> With that nothing changes in tdlib and tdesktop against tde2e just works.
> 
> Feedback? OK?

Build tested and the upgrade process went smoothly:
tdlib-1.8.44->1.8.49p0. I see no reason for the REVISION in
net/tdlib/tdlib. Otherwise OK rsadowski

> 
> Index: net/py-telegram/Makefile
> ===================================================================
> RCS file: /cvs/ports/net/py-telegram/Makefile,v
> diff -u -p -r1.11 Makefile
> --- net/py-telegram/Makefile	29 Apr 2025 10:39:07 -0000	1.11
> +++ net/py-telegram/Makefile	21 May 2025 05:25:33 -0000
> @@ -4,7 +4,6 @@ MODPY_DISTV =	0.19.0
>  
>  DISTNAME =	python_telegram-${MODPY_DISTV}
>  PKGNAME =	py-telegram-${MODPY_DISTV}
> -REVISION =	0
>  
>  CATEGORIES =	net
>  
> @@ -18,7 +17,7 @@ MODULES =	lang/python
>  MODPY_PI =	Yes
>  MODPY_PYBUILD =	setuptools_scm
>  
> -RUN_DEPENDS =	net/tdlib \
> +RUN_DEPENDS =	net/tdlib/tdlib>=1.8.49p0 \
>  		textproc/py-telegram-text
>  
>  # remove bundled binaries