Upstream name is lowercase(PyJWT) [1], match in PORTNAME

(1) https://pypi.python.org/pypi/PyJWT

Missing LICENSE/LICENSE_FILE

This upstream is good, since they set test_suite= in setup.py [1], which means python setup.py test command works out of the box to run tests.

Add

regression-test: build
<tab>@cd ${WRKSRC} && {$PYTHON_CMD} ${PYSETUP} test

(1) https://github.com/progrium/pyjwt/blob/master/setup.py#L32

Send this upstream (github PR/issue)

Send this upstream (github PR/issue)

Homepage is normally set by authors in setup.py under the URL= entry [1]. Use it unless there's a better option.

(1) https://github.com/progrium/pyjwt/blob/master/setup.py#L18

Missing WWW: ?

• PORTNAME=lowercase(upstream_name)
• py-pyjwt for svn dirname

Note: You'll probably need to set DISTNAME to get distfiles/fetching done once PORTNAME is lowercased

LICENSE and LICENSE_FILE please :) If not for the License Framework, then for Python Ports and Team. (because we care, together, as a team)

Personally speaking, my own opinions and feelings about liability or effectiveness of a particular feature don't preclude me from participating or warrant standing-off on that basis alone.

RSASSA-PKCS1-v1_5 signature support

"=" is sufficient for option_*_DEPENDS

LICENSE=MIT
LICENSE_FILE=${WRKSRC}/LICENSE

(1) https://github.com/progrium/pyjwt/blob/master/setup.py#L24
(2) https://github.com/progrium/pyjwt/blob/master/LICENSE

FWIW, adding LICENSE information is metadata (if available), not an assertion.

I do agree they should only be added if explicitly specified and unambiguous upstream

This override is not necessary anymore

s/-v1_5/ v1.5/ ?

I prefer *_DEPENDS to come before options (and iirc, portlint likes them earlier too), but this is a minor nit.

${PYDISTUTILS_SETUP} is better than ${PYSETUP} in what will become the canonical test command invocation.

It wraps the 'setup.py' string in a setuptools import, which adds the setuptools 'test' command to distutils only ports.