[NEW PORT] audio/py-opuslib: Python bindings to the libopus, IETF low-delay audio codec

[ultima]

Summary

Python bindings to the libopus, IETF low-delay audio codec.

WWW: https://github.com/onbeep/opuslib

PR\: 222038
Submitted by\: Yuri Victorovich (maintainer)
Reviewed by\: lifanov (mentor), matthew (mentor)
Approved by\: lifanov (mentor), matthew (mentor)
Differential Revision\: https://reviews.freebsd.org/DXXXXX

Test Plan

portlint:
looks fine.

poudriere:
103i386
103amd64
110i386
110amd64
12i386
12amd64

[koobs]

audio/py-opuslib/Makefile

20

setup.py contains a test_suite entry, so adding a test target to run unit tests is trivial, and should be added where tests are provided in the source distribution (ask upstream to include them if they're not)

do-test:
    @cd ${WRKSRC} && ${PYTHON_CMD} ${PYDISTUTILS_SETUP} test

[matthew]

New port needs entries in the Category Makefile. Other than that, and Koob's comment this is fine.

[ultima]

In D12309#255480, @matthew wrote:

New port needs entries in the Category Makefile. Other than that, and Koob's comment this is fine.

I was planning on using the Tools/scripts/addport script to add the port which automatically adds this bit. Is this acceptable or does it require to be in the Phabricator differential?

[ultima]

audio/py-opuslib/Makefile

20

Although there is a test_suite, the reason it is not in the port is due to not being provided by CHEESESHOP src like it does in github and will throw error:

ImportError: No module named tests

[koobs]

@ultima Thanks for clarifying. Not a blocker for commit, but ask upstream (or ask maintainer to ask upstream) to add tests/test files to the source distribution. This usually takes the form of adding entries to MANIFEST.in to include the files/dirs.

[koobs]

Approved, obviously pending SUBDIR addition to category/Makefile

[matthew]

[sunpoet]

audio/py-opuslib/Makefile

17

Please use USES=python:2.7 since we do not support any prior versions.

18

Please sort USE_PYTHON.

[lifanov]

Approved!

[mat]

audio/py-opuslib/pkg-descr

1

This is too short.

[ultima]

audio/py-opuslib/pkg-descr

1

I agree with you. I'm not really sure what else should be said though as this is all the package says upstream.

[ultima]

Addressed items

[matthew]