Created attachment 195215 [details] patch 0.31.2 to 0.32.2 I added a patch from py-matrix-synapse-0.31.2 to 0.32.2 "works for me"
At a first glance the patch indeed corrects some dependencies (thanks!) but does not add the one that actually gave rise to the new release (which should be something like ${PYTHON_PKGNAMEPREFIX}attrs>0:net-mgmt/py-attrs@${FLAVOR} in the rundeps if I'm not mistaken). I've added this and will give the resulting package a spin tomorrow. Cheers, Sascha
i ignored attrs as it's a dependency of other dependencies anyways. I'm not sure, what's best practice in such a case. root@riot:~# pkg info -r py27-attrs py27-attrs-18.1.0: py27-service_identity-17.0.0 py27-Automat-0.7.0
(In reply to Markus Mann from comment #2) Python ports should declare/match the dependencies declared by upstream, as closely as possible, without being incorrect. Upstream is the best and canonical source of this information. More broadly and specifically, a port should depend on exactly everything the software (code) uses *itself*. Whether or not another dependency of that software depends on it too or not is not a consideration. In summary, it either depends on it, or it doesn't. If it uses it, it depends on it. If spurious dependencies are identified (a declared dependency that is not actually used), then it should be reported and fixed upstream as a first course of action. *optionally* it can be not included, or removed, and/or commented as such in the port until the change is released in subsequent versions. However ... The support burden for not having dependencies (or anything else for that matter), exactly as declared by/in upstream code is on the person(s) making that decision, and since the spurious dependency downside is minimal in most cases (all else being equal), it is not worth doing (and maintaining that discrepancy) in favour of just correcting the issue upstream.
Created attachment 195255 [details] py-matrix-synapse 0.31.2 to 0.32.2 patch I've revised the patch a bit, added the py-attrs as well as the py-psutil dependency (missing from upstream deps) and moved the py-netaddr dependency to the "officially" listed deps. I'll probably need to take a look at the other "discovered" dependencies in this package and either confirm their necessity or remove them from the port, but for now this should do. In any case, the new version seems to work fine[tm].
@Sasche: You did a diff from 0.31.1 which is not the current version in the ports. Anyways, upstream already moved on to 0.33.0, so i made a new patch from the actually in the ports r.31.2 to 0.33.0 and incorporated your changes for 0.32.2. And it still works for me. Hopefully we can get this one applied now.
Created attachment 195506 [details] py-matrix-synapse 0.31.2 to 0.33.0
reopen
raised my patch to 0.33.1, a security patch https://github.com/matrix-org/synapse/commit/c2a83349f026c964302c6ad50a402c4cd664367f
Created attachment 195767 [details] py-matrix-synapse 0.31.2 to 0.33.1
Created attachment 195768 [details] py-matrix-synapse 0.31.2 to 0.33.1 patch Hi Markus, Sorry for the delay, your patch looks fine, but I think missed two FLAVOR -> PY_FLAVOR changes. I've changed this and bumped to 0.33.1 for the latest security fixes. Should be ok and works for me.
Created attachment 196024 [details] py-matrix-synapse 0.31.2 to 0.33.2 patch Another week, another release. This time a bugfix release [1]. I would usually avoid unnecessary bumps, but we should really get one of these things committed as there are both security fixes and important bugfixes included in the recent updates. In any case, the new version works fine on my matrix instance. Cheers, Sascha [1] https://github.com/matrix-org/synapse/releases/tag/v0.33.2
A commit references this bug: Author: decke Date: Thu Aug 9 11:29:38 UTC 2018 New revision: 476738 URL: https://svnweb.freebsd.org/changeset/ports/476738 Log: net-im/py-matrix-synapse: Update to 0.33.2 PR: 229846 Submitted by: Sascha Biberhofer <ports@skyforge.at> (maintainer) Reported by: Markus Mann <freebsd-public@it64.de> Changes: head/net-im/py-matrix-synapse/Makefile head/net-im/py-matrix-synapse/distinfo
Committed. Thanks!