[Bug 258593] [NEW] sysutils/py-puremagic: Pure python implementation of magic file detection
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
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.