Re: New port: net/qtox and databases/sqlcipher and an update to net/toxcore

On Fri, Jan 01, 2021 at 10:42:00PM +0000, Stuart Henderson wrote:
> On 2021/01/01 18:52, Tom Murphy wrote:
> > Hi,
> >
> > Last port tarball was missing the #define USE_V4L so I've updated
> > it so it works.
> >
> > However, I can't seem to test this. Anything that tries to open
> > /dev/video even with kern.video.record=1 seems to fail.
>
> re the problem opening video:
>
> did you try video(1) too or just things from ports? kern.video.record=1
> should not stop it from opening, it just zeroes the video buffers before
> passing the frames to userland, though I have not updated to a version
> with that yet.
>
> re v4l2 in this port:
>
> it might be better to figure out a bit more what's going on and either
> prove that it works or doesn't work and enable or not based on that.
> v4l applications on OpenBSD often need some other changes than just the
> headers, I've had to workaround a problem with video ioctls failing on a
> few other ports.

I tried video(1) and it works on my system, so it's probably an ioctl.
I'll try to find the cause, but aside from the video device not working,
the rest of the qtox port works. I tested an audio call with a friend
and it worked fine. Also the messaging and other features work. It's
just the video that isn't working yet.

>
> > I'm on:
> >
> > OpenBSD 6.8-current (GENERIC.MP) #248: Tue Dec 29 12:07:27 MST 2020
> >
> > Anyway, it should properly try to open the video device now.
> >
> > Updated port attached as qtox3.tgz here.
> >
> > Thanks,
> > Tom
>
> couple of port tweaks below, I haven't looked closer yet.
>
>
> diff 4a421e6a1869cd82df717c62298316607805167d /usr/ports/mystuff
> blob - 5c7e9fee75b3ea2e473fb1d28a0e3274821b5ed5
> file + net/qtox/Makefile
> --- net/qtox/Makefile
> +++ net/qtox/Makefile
> @@ -1,20 +1,18 @@
> # $OpenBSD: Makefile,v 1.7 2020/04/05 09:45:30 ajacoutot Exp $
>
> -N = qtox
> -V = 1.17.3
> COMMENT = Tox client
>
> -PKGNAME = ${N}-${V}
> GH_ACCOUNT = qTox
> GH_PROJECT = qTox
> -GH_TAGNAME = v${V}
> +GH_TAGNAME = v1.17.3
> +PKGNAME = ${DISTNAME:L}
>
> CATEGORIES = net
>
> HOMEPAGE = https://qtox.github.io/
> MAINTAINER = Tom Murphy <openbsd@pertho.net>
>
> -# GPLv3
> +# GPLv3+
> PERMIT_PACKAGE = Yes
>
> WANTLIB += ${COMPILER_LIBCXX} KF5SonnetUi Qt5Core Qt5Gui Qt5Network
>

Tweaks look good. I guess all the ${V} and ${N} stuff isn't needed
any more -- looks a lot cleaner, thanks!

Tom