[Bug 258593] [NEW] sysutils/py-puremagic: Pure python implementation of magic file detection

From: <bugzilla-noreply_at_freebsd.org>
Date: Mon, 20 Sep 2021 00:56:21 UTC
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=258593

Kubilay Kocak <koobs@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[New Port]                  |[NEW]
                   |sysutils/py-puremagic       |sysutils/py-puremagic: Pure
                   |Module that will identify a |python implementation of
                   |file based off it's magic   |magic file detection
                   |numbers.                    |
                 CC|                            |james@french.id.au,
                   |                            |python@FreeBSD.org
           Keywords|                            |feature, needs-patch,
                   |                            |needs-qa
             Status|New                         |Open
              Flags|                            |maintainer-feedback?(james@
                   |                            |french.id.au)

--- Comment #2 from Kubilay Kocak <koobs@FreeBSD.org> ---
Thanks for submitting a new port James.

Review items:

- Add LICENSE_FILE when a license file is provided with a distribution file.
This one does: LICENSE

- Match COMMENT to upstream (setup.py:description). In this case "Pure python
implementation of magic file detection"

- Ports must match and include all upstream dependency declarations. In this
case, upstream defines:  install_requires=["argparse"], which *can* be provided
by the stdlib, or not. Add a RUN_DEPENDS for this

- Add TEST_DEPENDS and a do-test target when upstream ships tests with their
distfiles. This one does: pytest for tests and the following for an invocation
should work:

  cd ${WRKSRC} && ${PYTHON_CMD}-m pytest -v -rs -o addopts= test/

-o addopts= overrides upstream pytest configurations that often require
coverage and other dev only tests which we dont need (this one does).

See Also: https://wiki.freebsd.org/Python/PortsPolicy

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