https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=204162
Kubilay Kocak <ko...@freebsd.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |feature, needs-patch,
| |needs-qa, patch
Summary|New port: devel/py-docker |[NEW PORT]
|An API client for docker |devel/py-docker:API client
|written in Python |for docker written in
| |Python
CC| |ko...@freebsd.org,
| |pyt...@freebsd.org
--- Comment #1 from Kubilay Kocak <ko...@freebsd.org> ---
@Uros, thanks for your submission :)
On initial review, at least the following changes are needes:
* Remove 'An' (indefinite article from COMMENT. Use the upstream
(setup.py:description) instead. "Python client for Docker"
* Rename to (dir, portname) to docker-py (as per PyPI name):
https://pypi.python.org/pypi/docker-py
For more info, see: https://wiki.freebsd.org/Python/PortsPolicy
* RUN_DEPENDS no longer need to be added to TEST_DEPENDS.
* Do the requirements in setup.py for RUN_DEPENDS use '>' or '>=' ? Please
update them if they are >=.
* Add 'python' as a secondary (virtual) CATEGORIES
* Remove GH_PROJECT, wont be needed after PORTNAME change above.
* Remove all commented out variables/targets, unless they are comments. If they
are works in progress, don't commit them.
* Rename regression-test: build target to 'do-test:'. This is a new target (as
part of a new test framework) that was recently committed
* Use the PYDISTUTILS_SETUP variable in your test invocation, as it is
correctly wrapped in a setuptools import command.
Please also confirm that this port passes QA (portlint -AC, poudriere testport)
--
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
freebsd-python@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-python
To unsubscribe, send any mail to "freebsd-python-unsubscr...@freebsd.org"
