From: Kirill A. Korinsky <kirill@korins.ky>
Subject: Re: [new port] productivity/remind-caldav
To: Martin Ziemer <horrad@horrad.de>
Cc: ports@openbsd.org
Date: Mon, 23 Jun 2025 20:50:22 +0200

On Mon, 23 Jun 2025 06:38:53 +0200,
Martin Ziemer <horrad@horrad.de> wrote:
> 
> Am Sun, Jun 22, 2025 at 03:40:36PM +0200 schrieb Kirill A. Korinsky:
> > On Sun, 22 Jun 2025 13:12:42 +0200,
> > Martin Ziemer <horrad@horrad.de> wrote:
> > > Am Sun, Jun 22, 2025 at 10:24:09AM +0200 schrieb Kirill A. Korinsky:
> > > > On Fri, 20 Jun 2025 09:06:47 +0200,
> > > > Both new ports read ok for me, but make test at py-remind fails as:
> > > Found the Error: there was no dependency on py-vobject, which is needed.
> > > (On my system this package was already installed, so no visible error)
> > > Added dependency to py-dateutil as well, as it is named in setup.py.
> > You have in py-remind:
> > 
> > BUILD_DEPENDS =	devel/py-coverage \
> > 		devel/py-nose \
> > 		textproc/py-vobject \
> > 		devel/py-dateutil
> > 
> > But https://github.com/jspricke/python-remind/blob/v0.19.2/setup.py has:
> > 
> > install_requires=["python-dateutil", "vobject"],
> > 
> > are you sure that it is BUILD_DEPENDS and not RDEPS?
> > 
> > also, I don't see any trace of nose nor coverage which seems was droped by
> > https://github.com/jspricke/python-remind/commit/5d788c226cb962341092acff02957b5dcb3e559e
> You are right. And looking into it the Depends on remind-caldav itself
> were also outdated.
> I Fixed Depends for both in the attached tarball

With one minor nickpik:

py-remind $ diff -uNar pkg/PLIST* 
--- pkg/PLIST   Mon Jun 23 20:39:12 2025
+++ pkg/PLIST.orig      Mon Jun  2 15:02:16 2025
@@ -1,5 +1,6 @@
 bin/ics2rem
 bin/rem2ics
+lib/python${MODPY_VERSION}/site-packages/${MODPY_PYCACHE}/
 lib/python${MODPY_VERSION}/site-packages/${MODPY_PYCACHE}ics_compare.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
 lib/python${MODPY_VERSION}/site-packages/${MODPY_PYCACHE}ics_compare.${MODPY_PYC_MAGIC_TAG}pyc
 lib/python${MODPY_VERSION}/site-packages/${MODPY_PYCACHE}remind.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
py-remind $

I think that it worth to be imported.

OK kirill@ to import both ports.

-- 
wbr, Kirill