Re: [UPDATE] sysutils/py-daemon 1.6 -> 2.2.4

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@