From: Theo Buehler <tb@theobuehler.org>
Subject: Re: [new] 17 new ports needed for apache superset
To: Daniel Dickman <didickman@gmail.com>
Cc: ports@openbsd.org
Date: Fri, 12 Dec 2025 11:23:26 +0100

I think this is almost ready to go in (see comment to py-flask-compress).

Some replies and new comments for the remainder of the deps:

> > databases/py-flask-migrate
> > 
> > I would add
> > 
> > TEST_DEPENDS = ${FULLPKGNAME}:${BUILD_PKGPATH} \
> > 		databases/py-flask-sqlalchemy
> 
> --> This was actually a missing RDEP; now added thanks.

right. I'm pretty sure the tdep on self is needed.

> > databases/py-pgsanity
> > 
> > missing tdep on databases/postgresql
> 
> --> should not be needed?
> --> Should already be an RDEP?

There's no tdep on self, so the port isn't necessarily installed when
you run tests and since it's an rdep, postgresql may be missing. Either
you need a tdep on self or on postgresql.

> > NO_TEST = Yes
> 
> --> I may leave these out in case tests are ever added to the pypi 
> --> tarballs.

up to you

> > security/py-google-auth
> 
> -> was missing TDEPs on mock/freezegun/aioresponses; added

Right. There's the oddity that one id_token test fails unless you have
py3-jwt installed. The failing mtls_helper test seems to be written for
an older pyopenssl.

> > security/py-pyu2f

missing rdep on six, should have tdep on unittest, mock and an annotation
for missing fakefs

> > sysutils/py-cron-descriptor

I believe COMMENT and DESCR want a hyphen in "human-readable"

> > www/py-flask-compress

For py313 this needs backports.zstd as an rdep/tdep. I'll send my wip in
a separate mail.

> > www/py-flask-session
> >