Page MenuHomeFreeBSD
Differential D24654 Diff 71245 sysutils/py-mqttwarn/Makefile

Changeset View

Standalone View

View Options

sysutils/py-mqttwarn/Makefile

# $FreeBSD$ # $FreeBSD$
PORTNAME= mqttwarn PORTNAME= mqttwarn
DISTVERSION= 0.10.4 DISTVERSION= 0.15.0
PORTREVISION= 1
CATEGORIES= sysutils python CATEGORIES= sysutils python
PKGNAMEPREFIX= ${PYTHON_PKGNAMEPREFIX} PKGNAMEPREFIX= ${PYTHON_PKGNAMEPREFIX}
MAINTAINER= dvl@FreeBSD.org MAINTAINER= dvl@FreeBSD.org
COMMENT= Subscribe to MQTT topics and notify pluggable services COMMENT= Subscribe to MQTT topics and notify pluggable services
LICENSE= EPL LICENSE= EPL
LICENSE_FILE= ${WRKSRC}/LICENSE LICENSE_FILE= ${WRKSRC}/LICENSE
DEPRECATED= Uses deprecated version of python RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}six>0:devel/py-six@${PY_FLAVOR}
koobsUnsubmitted
Done
Inline Actions

This can be dropped if it now supports versions other than 2.7

koobs: This can be dropped if it now supports versions other than 2.7
koobsUnsubmitted
Not Done
Inline Actions

RUN_DEPENDS <version-specs> should always match those in setup.py as closely as possible:

+    'six>=1.13.0',
+    'paho-mqtt>=1.4.0',
+    'jinja2>=2.10.1',
+    'attrs>=19.3.0',
+    'docopt>=0.6.2',
+    'requests>=2.22.0',
+    'future>=0.18.2',
+    'configparser>=3.5.3',
koobs: RUN_DEPENDS <version-specs> should always match those in setup.py as closely as possible: ```…
EXPIRATION_DATE= 2020-08-15
RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}paho-mqtt>=0:net/py-paho-mqtt@${PY_FLAVOR} RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}paho-mqtt>=0:net/py-paho-mqtt@${PY_FLAVOR}
RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}Jinja2>0:devel/py-Jinja2@${PY_FLAVOR}
RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}attrs>=19.3.0:devel/py-attrs@${PY_FLAVOR}
RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}docopt>=0.6.2:devel/py-docopt@${PY_FLAVOR}
RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}requests>=2.22.0:www/py-requests@${PY_FLAVOR}
RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}future>=0.18.2:devel/py-future@${PY_FLAVOR}
RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}configparser>0:devel/py-configparser@${PY_FLAVOR}
RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}supervisor>=0:sysutils/py-supervisor@${PY_FLAVOR} RUN_DEPENDS+= ${PYTHON_PKGNAMEPREFIX}supervisor>=0:sysutils/py-supervisor@${PY_FLAVOR}
koobsUnsubmitted
Done
Inline Actions

Can we collapse these lines into a single RUN_DEPENDS with \ line endings

koobs: Can we collapse these lines into a single RUN_DEPENDS with \ line endings
USES= python:2.7 shebangfix USES= python:3.6+ shebangfix
USE_PYTHON= flavors USE_PYTHON= autoplist distutils
koobsUnsubmitted
Done
Inline Actions

Why does this need flavors?

koobs: Why does this need flavors?
dvlAuthorUnsubmitted
Done
Inline Actions

I cannot recall now. Removed.

dvl: I cannot recall now. Removed.
koobsUnsubmitted
Done
Inline Actions

Ports should 'declare' the versions that a package supports and not unnecessarily restrict/limit them.

The package support 2.7. USES=python is appropriate here.

koobs: Ports should 'declare' the versions that a package supports and not unnecessarily…
koobsUnsubmitted
Done
Inline Actions

cov (coverage) and (tox) are upstream development related test dependencies, and not necessary for downstream testing.

  • Instead of depending on tox, depend on the thing that tox runs. In this case, only pytest [1]
  • To remove/override non-compulsory upstream development test dependencies, invoke pytest directly in the do-test target:
do-test:
		@cd ${WRKSRC} && ${PYTHON_CMD} -m pytest -v -rs -o addopts=

-o addopts= overrides test settings/arguments set in pytest.ini or setup.cfg:[pytest] sections (which adds --cov, requiring pytest-cov)

Upstream should move these 'non-compulsory' test dependencies to an extras_require[dev] section, that tox.ini then pulls in. This will leave extras_require[test] as purely compulsory test dependencies (pytest, and optional run time deps)

[1] https://github.com/jpmens/mqttwarn/blob/master/tox.ini#L8

koobs: cov (coverage) and (tox) are upstream development related test dependencies, and not necessary…
SHEBANG_REGEX= '.*\.py$$' SHEBANG_REGEX= '.*\.py$$'
USE_GITHUB= yes USE_GITHUB= yes
GH_ACCOUNT= jpmens GH_ACCOUNT= jpmens
NO_ARCH= yes NO_ARCH= yes
OPTIONS_DEFINE= DOCS EXAMPLES OPTIONS_DEFINE= DOCS EXAMPLES
USERS= mqttwarn USERS= mqttwarn
GROUPS= mqttwarn GROUPS= mqttwarn
SUB_FILES= pkg-message SUB_FILES= pkg-message
post-patch: post-patch:
@${REINPLACE_CMD} -e 's|MQTTWARN_LIBDIR|${PYTHON_SITELIBDIR}/${PORTNAME}/lib/|g' ${WRKSRC}/mqttwarn.ini.sample @${REINPLACE_CMD} -e 's|MQTTWARN_LIBDIR|${PYTHON_SITELIBDIR}/${PORTNAME}/lib/|g' ${WRKSRC}/mqttwarn/examples/basic/mqttwarn.ini
do-build: #do-build:
(cd ${WRKSRC} && ${PYTHON_CMD} -m compileall .) # (cd ${WRKSRC} && ${PYTHON_CMD} ${PYTHON_LIBDIR}/compileall.py .)
koobsUnsubmitted
Not Done
Inline Actions

All of these extras_require (optional deps) need to be switched from == to >= too.

Note: this does not guarantee that upstream tests with them, or that they will work.

Upstream needs to change their pinned (==) deps to >= (for sdist/setup.py/users) so that they are testing future versions of their dependencies before users get them. This also avoids conflicts between package versions downstream as well.

If they want to continue to pin versions, this can be done in requirements.txt files (unrelated to distribution), but this is not recommended either.

koobs: All of these extras_require (optional deps) need to be switched from == to >= too. Note: this…
do-install: do-install:
(cd ${WRKSRC}/build && ${PYTHON_CMD} ${PYTHON_LIBDIR}/compileall.py .)
${MKDIR} ${STAGEDIR}/var/log/${PORTNAME} ${MKDIR} ${STAGEDIR}/var/log/${PORTNAME}
${INSTALL_SCRIPT} ${WRKSRC}/mqttwarn.py ${STAGEDIR}${PREFIX}/sbin/mqttwarn ${INSTALL_SCRIPT} ${WRKSRC}/mqttwarn ${STAGEDIR}${PREFIX}/sbin/mqttwarn
koobsUnsubmitted
Done
Inline Actions

is this all that's required? There's no Python mysql package required to wrap mysqlclient.so ?

koobs: is this all that's required? There's no Python mysql package required to wrap mysqlclient.so ?
(cd ${WRKSRC}/etc && ${COPYTREE_SHARE} . ${STAGEDIR}${ETCDIR}) (cd ${WRKSRC}/etc && ${COPYTREE_SHARE} . ${STAGEDIR}${ETCDIR})
(cd ${WRKSRC}/services && ${COPYTREE_SHARE} . ${STAGEDIR}${PYTHON_SITELIBDIR}/${PORTNAME}/lib/services) (cd ${WRKSRC}/services && ${COPYTREE_SHARE} . ${STAGEDIR}${PYTHON_SITELIBDIR}/${PORTNAME}/lib/services)
${CP} ${WRKSRC}/mqttwarn.ini.sample ${STAGEDIR}${ETCDIR} @${REINPLACE_CMD} 's|%%PYTHONPREFIX_SITELIBDIR%%|${PYTHONPREFIX_SITELIBDIR}|g' ${WRKSRC}/mqttwarn/examples/basic/mqttwarn.ini
${CP} ${WRKSRC}/mqttwarn/examples/basic/mqttwarn.ini ${STAGEDIR}${ETCDIR}/mqttwarn.ini.sample
do-install-DOCS-on: do-install-DOCS-on:
@${MKDIR} ${STAGEDIR}${DOCSDIR} @${MKDIR} ${STAGEDIR}${DOCSDIR}
${INSTALL_DATA} ${WRKSRC}/README.md ${STAGEDIR}${DOCSDIR} ${INSTALL_DATA} ${WRKSRC}/README.md ${STAGEDIR}${DOCSDIR}
do-install-EXAMPLES-on: do-install-EXAMPLES-on:
@${MKDIR} ${STAGEDIR}${EXAMPLESDIR} @${MKDIR} ${STAGEDIR}${EXAMPLESDIR}
(cd ${WRKSRC}/examples && ${COPYTREE_SHARE} . ${STAGEDIR}${EXAMPLESDIR}) (cd ${WRKSRC}/examples && ${COPYTREE_SHARE} . ${STAGEDIR}${EXAMPLESDIR})
.include <bsd.port.mk> .include <bsd.port.mk>