06.04.2026 17:31, mykyta@nazarenko.sbs пишет: > Fixed. > > Patch with pledge(2) removed, but I think I will add it in the future, ofc with unveil(2), comments and explanation. > > But idk what to do in relation to tdlib, because ( as mentioned in Makefile ), a certain version of the library is tied to a certain version of nchat. The developer himself told me about this problem. So I'm thinking it's not possible. Looks liks using our tdlib isn't possible without patching, anyways, so I'll just ignore that and stick with what upstream does. - Your comment about i386 now explains why you limited support to two archs, but the port should just disable i386 explicitly (and more). - ccache needs proper disabling; USE_CCACHE is a user-setting and bulk builds run without it. Having fixed that, it seems you are not building with PORTS_PRIVSEP=Yes, another requirement for bulks and strong recommendation to anyone. - Similar thing with Git; I used the right CMake knob, but I am not sure, whether the build system still makes use of it via scripts, etc. - Take a look at 'make configure output' and what's in WRKBUILD before `make build` to see what other dependencies are required and/or picked up if present at build time. - I added libpng, apparently used for clipboard/image support. - The final linker invocation only shows internal libraries being linked statically, all external ones are dynamically linked as confirmed by `make port-lib-depends-check`. - I took the liberty to shorten COMMENT and DESCR a bit. Below diff incorporates these changes and packages for me, however, I might have missed something and also do not plan to use this myself, so if this should be imported (before release), review and testing from another developer using nchat would be appreciated. diff --git a/net/nchat/Makefile b/net/nchat/Makefile index a63819e5b..9c1f2aef1 100644 --- a/net/nchat/Makefile +++ b/net/nchat/Makefile @@ -1,41 +1,37 @@ -# i386 is supported -# https://github.com/d99kris/nchat/issues/76 -ONLY_FOR_ARCHS = amd64 arm64 i386 +# see https://github.com/d99kris/nchat/issues/76 and net/tdlib (bundled) +NOT_FOR_ARCHS = i386 ${BE_ARCHS} -COMMENT = CLI-based Telegram client +# no WhatsApp or Signal: https://github.com/d99kris/nchat/commit/f292853 +COMMENT = CLI Telegram client GH_ACCOUNT = d99kris GH_PROJECT = nchat GH_TAGNAME = v5.14.44 CATEGORIES = net -HOMEPAGE = https://github.com/d99kris/nchat MAINTAINER = Mykyta Nazarenko <mykyta@nazarenko.sbs> -# note : nchat using it's own version of tdlib due to no proper versioned releases of this library -# it may be difficult to keep API compatibility if using tdlib as a separate package ( net/tdlib ) -WANTLIB += sqlite3 magic - -DPB_PROPERTIES = parallel +WANTLIB += ${COMPILER_LIBCXX} c crypto curses execinfo m magic +WANTLIB += png sqlite3 ssl z # nchat: MIT # tdlib: BSLv1.0 PERMIT_PACKAGE = Yes -# at the moment nchat does not support WhatsApp and Signal on OpenBSD, static-linking with cgo is restricted -# https://github.com/d99kris/nchat/commit/f292853 - # C++17 COMPILER = base-clang ports-gcc -USE_CCACHE = Yes - MODULES = devel/cmake BUILD_DEPENDS = devel/gperf LIB_DEPENDS = databases/sqlite3 \ - devel/libmagic + devel/libmagic \ + graphics/png + +CONFIGURE_ARGS = -DCCACHE_FOUND=OFF \ + -DCCACHE_PROGRAM=OFF \ + -DCMAKE_DISABLE_FIND_PACKAGE_Git=ON NO_TEST = Yes diff --git a/net/nchat/pkg/DESCR b/net/nchat/pkg/DESCR index 05edcfadf..20c986c03 100644 --- a/net/nchat/pkg/DESCR +++ b/net/nchat/pkg/DESCR @@ -1,3 +1,2 @@ -nchat is a multi-protocol terminal-based messaging client for Linux and macOS -with support for Telegram, WhatsApp and Signal. Currently, WhatsApp and Signal -protocols are not supported on OpenBSD. +nchat is a multi-protocol terminal-based messaging client for Telegram, WhatsApp +and Signal, altough only the former is currently supported on OpenBSD.
No comments:
Post a Comment