Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 20 Sep 2021 00:56:21 +0000
From:      bugzilla-noreply@freebsd.org
To:        ports-bugs@FreeBSD.org
Subject:   [Bug 258593] [NEW] sysutils/py-puremagic: Pure python implementation of magic file detection
Message-ID:  <bug-258593-7788-OMtEGGWmuf@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-258593-7788@https.bugs.freebsd.org/bugzilla/>
References:  <bug-258593-7788@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D258593

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 pyth=
on
implementation of magic file detection"

- Ports must match and include all upstream dependency declarations. In this
case, upstream defines:  install_requires=3D["argparse"], which *can* be pr=
ovided
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 invocat=
ion
should work:

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

-o addopts=3D 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

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-258593-7788-OMtEGGWmuf>