Re: CVS commit: pkgsrc/chat/matrix-synapse

[Adam <adam%netbsd.org@localhost>]

>>> -DEPENDS+= ${PYPKGPREFIX}-idna>=2.5:../../www/py-idna
>>> -DEPENDS+= ${PYPKGPREFIX}-lxml>=3.5.0:../../textproc/py-lxml
>>> -DEPENDS+= ${PYPKGPREFIX}-nacl>=1.2.1:../../security/py-nacl
>> These show up in the sources.  Why did you remove them?
> 
> +1, unclear why those were removed. Did you try if it even runs without those installed on the system?

idna is only needed for testing (tests/config/test_tls.py)
lxml is an optional dependency for "url-preview" (see pyproject.toml)
nacl is not present in the source code, nor in pyproject.toml; it's required by py-macaroons and py-signedjson, so it gets installed anyway.

>>> -DEPENDS+= ${PYPKGPREFIX}-psycopg2>=2.7:../../databases/py-psycopg2
>> Why is this dropped?  synapse needs a postgres database and we as
>> maintainers have more or less given up on sqlite3.
> 
> Please don't drop this. sqlite3 isn't only given up on by us - it's by upstream as well. To the point where if you use sqlite3 and update Synapse, there is no guarantee that your DB is still 
> intact. It only exists for testing purposes and should never be used in any use case that would be covered by pkgsrc. It's only for synapse development itself at this point, where you don't mind 
> just wiping the entire DB regularly.

I've restored psycopg2, lxml and authlib, and added idna for testing. Shoot if that's wrong. :)

Kind regards,
Adam