On Wed 26/02/2020 11:32, Martin Reindl wrote: > On Wed, Feb 26, 2020 at 11:02:32AM +0100, Bjorn Ketelaars wrote: > > On Wed 26/02/2020 10:15, Martin Reindl wrote: > > > Update py-daemon from 1.6 to 2.2.4: > > > > > > - move from python2 to python3-only (no consumers) > > > - take MAINTAINER > > > - ' =' vs '=' sanity in Makefile > > > - license changed to GPL-3/ASF-2 > > > - adjust tests, needs py-testscenarios which I might send out at a later > > > point > > > 1 failed, 242 passed, 2 skipped, 1 warnings > > > - sending this out now so that any users can speak up if they need/want > > > the > > > python2 variant > > > - extra adjustment in quirks not included in the diff > > > > > > Comments? OK? > > > > > > +CATEGORIES = sysutils > > > > > > -CATEGORIES= sysutils > > > > Maybe you could add HOMEPAGE=https://pagure.io/python-daemon/ ? > > > > > +BUILD_DEPENDS = www/py-requests${MODPY_FLAVOR} \ > > > + textproc/py-docutils${MODPY_FLAVOR} \ > > > + security/py-keyring${MODPY_FLAVOR} > > > > > > This update seems to need twine. I'm building with PORTS_PRIVSEP=Yes and > > I'm seeing the error below: > > > > distutils.errors.DistutilsError: Could not find suitable distribution for > > Requirement.parse('twine') > > Download error on https://pypi.org/simple/twine/: [Errno 65] No route to > > host -- Some packages may not be found! > > Couldn't find index page for 'twine' (maybe misspelled?) > > > > I glanced at the documentation of py-daemon and didn't find any > > mentioning of py-keyring. Why is this needed as BDEP? > > Twine needs py-keyring. But twine is only needed for publishing to PyPI. > Fixed diff below plus patches/ and HOMEPAGE.
Diff looks good and build tested. OK bket@
