[Bug 243967] [NEW PORT] net-p2p/bazarr: Automatic subtitle downloader for Sonarr and Radarr

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=243967

Kubilay Kocak <koobs at FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |needs-patch, needs-qa
                 CC|                            |koobs at FreeBSD.org,
                   |                            |michiel at vanbaak.eu,
                   |                            |python at FreeBSD.org
              Flags|                            |maintainer-feedback?(michie
                   |                            |l at vanbaak.eu)

--- Comment #1 from Kubilay Kocak <koobs at FreeBSD.org> ---
Review items:

- Upstream supports [1][2] Python 3.6+ (testing up to 3.8).
USES=python:<version-spec> should be 3.6+ or 3.6-3.8

- lxml RUN_DEPENDS version specifier should match the version specified in
upstream requirements.txt (>=4.3.0)

- All importable .py files need to be optimized/processed (pyc/pyo files).
Ports using distutils (setuptools) and autoplist handle this automatically.
This packages doesnt use setuptools (ask them to package and ship this via PyPI
please).

Not doing so means that the pyc/pyo files will be created at run time, thereby
leaving pyc/pyo orphaned files leftover after pkg removal (since they're not
referenced in the pkg-plist)

Use a post-install: Python "compileall" script (provided examples/references
via IRC) to process all .py files and include them in the pkg-plist.

Note: the net-p2p/couchpotato port, which this port was based on, is also
affected by this (only packaging .py files)

[1] "bazarr require Python 3.6 or greater and can be run from source."
[2] "Install Python 3.6 or greater (latest is good)"

-- 
You are receiving this mail because:
You are on the CC list for the bug.