Re: NEW: geo/py-planet

# $OpenBSD$

COMMENT = small addon for python-requests HTTP library

MODPY_EGG_VERSION = 0.9.7
DISTNAME = requests-futures-${MODPY_EGG_VERSION}
PKGNAME = py-${DISTNAME}

CATEGORIES = www

HOMEPAGE = https://github.com/ross/requests-futures

# APLv2
PERMIT_PACKAGE_CDROM = Yes

MODULES = lang/python

MODPY_PI = Yes
MODPY_SETUPTOOLS = Yes

FLAVORS = python3
FLAVOR ?=

RUN_DEPENDS += www/py-requests${MODPY_FLAVOR}

.if ! ${FLAVOR:Mpython3}
RUN_DEPENDS += devel/py-futures${MODPY_FLAVOR}
.endif

do-test:
cd ${WRKSRC} && ${MODPY_BIN} test_requests_futures.py


.include <bsd.port.mk>
On Mon, 29 May 2017 11:47:18 +0200, "Sebastian Reitenbach"
<sebastia@l00-bugdead-prods.de> wrote:

> This is a new port of the Planet API client library that also includes
> a command line tool. py-requests-futures is a needed depdendency.
>
> cat py-planet/pkg/DESCR
> This library provides a command-line-interface (CLI) and Python
> library to make access to Planet's public API easy to use.
>
> cat py-requests-futures/pkg/DESCR
> Small add-on for the python requests http library. Makes use of python
> 3.2's concurrent.futures or the backport for prior versions of python.
>
> The additional API and changes are minimal and strives to avoid
> surprises.
>
> both build for me with python3 flavor, only tested the default python
> 2.7
>
> Comments, OK?

Hi,

Here's a new Makefile attached and a new PLIST for py-requests-futures

For the PLIST: it's the ${MODPY_PYCACHE} dir that needs the
${MODPY_COMMENT}l

For the Makefile:
- reorder
- it's RUN_DEPENDS not LIB_DEPENDS I think
- take the usual FLAVOR check
- add a test target (py2 test didn't work otherwise)

With that ok danj@ for py-requests-futures.

I quickly looked at py-planet and the usual
FLAVORS = python3
FLAVOR ?=
isn't there. Can you verify the port following the review of
py-requests-futures please?

Cheers,
Daniel