Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 31 Oct 2015 06:33:37 +0000
From:      bugzilla-noreply@freebsd.org
To:        python@FreeBSD.org
Subject:   [Bug 204162] [NEW PORT] devel/py-docker:API client for docker written in Python
Message-ID:  <bug-204162-21822-gEfGxmLvqX@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-204162-21822@https.bugs.freebsd.org/bugzilla/>
References:  <bug-204162-21822@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=204162

Kubilay Kocak <koobs@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|                            |koobs@FreeBSD.org,
                   |                            |python@FreeBSD.org

--- Comment #1 from Kubilay Kocak <koobs@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.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-204162-21822-gEfGxmLvqX>