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
