On Wed 18/10/2017 21:18, Stuart Henderson wrote: > On 2017/10/18 21:56, Remi Locherer wrote: > > On Wed, Oct 18, 2017 at 07:51:47AM -0400, Daniel Jakots wrote: > > > On Mon, 16 Oct 2017 13:40:27 +0200, Björn Ketelaars > > > <[email protected]> wrote: > > > > > > > attached port > > > > > > In setup.py there is > > > install_requires=['Sphinx>=1.2b1'], > > > > > > shouldn't it need a RDEP then? > > > > > > borgbackup (the reason for guzzle theme) has a BDEP on py-sphinx and > > because of that it works. Maybe py-sphinx_guzzle_theme should have this > > RDEP instead of borgbackup. > > if py-sphinx_guzzle_theme has install_requires on sphinx, then sphinx should > be listed as a RUN_DEPENDS. >
I agree. Included in the attached port is the diff below...OK?
diff --git textproc/py-sphinx_guzzle_theme/Makefile
textproc/py-sphinx_guzzle_theme/Makefile
index cae7c9c7edc..1ff66a6da6c 100644
--- textproc/py-sphinx_guzzle_theme/Makefile
+++ textproc/py-sphinx_guzzle_theme/Makefile
@@ -15,6 +15,8 @@ PERMIT_PACKAGE_CDROM = Yes
MODULES = lang/python
+RUN_DEPENDS = textproc/py-sphinx${MODPY_FLAVOR}
+
MODPY_PI = Yes
MODPY_SETUPTOOLS = Yes
py-sphinx_guzzle_theme.tgz
Description: application/tar-gz
