Hi Stuart!

Thanks a lot for your feedback, much appreciated!

Below, please find a new port version with your feedback addressed.

Keep in mind that this still requires the infrastructure/db/user.list
change outlined in the original mail. You can also obtain the diff via
GitHub: https://github.com/nmeum/ports/compare/master..xandikos

Greetings
Sören

Stuart Henderson <stu@spacehopper.org> wrote:
> we are currently preparing for 7.7 release and not doing usual commits,
> but a few comments based on reading your port in the meantime.
> 
> On 2025/04/07 19:35, Sören Tempel wrote:
> > Hello!
> > 
> > Attached, please find a port for xandikos, which is a minimal,
> > Git-backed CalDav server. It offers high compatibility with various
> > CalDav standards, but does not support a multi-user setup or CalDAV
> > scheduling extensions.
> > 
> > In my view, it is more minimal and yet more standard-conforming than the
> > existing radicale2 port and should thus be considered for inclusion in
> > the ports tree.
> 
> In general: seems reasonable to add this software and I think only a few
> changes needed.
> 
> pkg/README:
> : xandikos listens by default on localhost:8080 without authentication.
> 
> Would be useful to show how to change the port - is it via flags or
> config file?
> 
> Maybe repeat the fact that it only supports a single user account per
> daemon instance here too, in case someone didn't understand the "not
> nulti-user" from DESCR.
> 
> : MODPY_DISTV =	0.2.12
> : DISTNAME =	xandikos-${MODPY_DISTV}
> : DISTFILES =	${DISTNAME}{v${MODPY_DISTV}}.tar.gz
> : PKGNAME =	${DISTNAME:L}
> ...
> : SITES =		https://github.com/jelmer/xandikos/archive/
> 
> don't use SITES with a github /archive/ URL.
> 
> is there a particular reason to use the github git-archive?
> 
> - if not, the pypi distfile is normally preferred (set MODPY_PI=Yes,
> remove DISTFILES, also the PKGNAME is not useful as the default setting
> works - :L is "lowercase this var" which is already lower case - so
> remove PKGNAME too). the distfile will differ so "make clean=dist" and
> "make makesum".
> 
> - if there is (e.g. if tests are only in the github one) then instead
> drop SITES, DISTFILES, DISTNAME, PKGNAME, add GH_ACCOUNT=jelmer,
> GH_PROJECT=xandikos, GH_TAGNAME=v${MODPY_DISTV}.
> 
> : REVISION =	0
> 
> no REVISION in a new port
> 
> : SUBST_VARS =	MODPY_SITEPKG
> 
> no need to set this, make show=SUBST_VARS will show that it's already
> included.
> 
> : RUN_DEPENDS =	devel/py-dulwich${MODPY_FLAVOR} \
> : 		www/py-aiohttp${MODPY_FLAVOR} \
> : 		textproc/py-defusedxml${MODPY_FLAVOR} \
> : 		devel/py-icalendar${MODPY_FLAVOR} \
> : 		www/py-jinja2${MODPY_FLAVOR} \
> : 		textproc/py-vobject${MODPY_FLAVOR}
> 
> we stopped using ${MODPY_FLAVOR} after 7.6.
> 
> : MODPY_PYTEST =	Yes
> 
> set by default, can be dropped
> 
> I have not yet run it, only read through.

• xandikos.tgz (3K)