Re: NEW: textproc/py-sphinx_guzzle_theme

[Daniel Jakots]

On Thu, 19 Oct 2017 07:19:27 +0200, Björn Ketelaars
 wrote:

Thanks! I just imported it with a minor tweak

> MODULES = lang/python
>
> +RUN_DEPENDS =textproc/py-sphinx${MODPY_FLAVOR}
> +
>  MODPY_PI =   Yes
>  MODPY_SETUPTOOLS =   Yes

I put the RDEP line after the MODPY lines to follow the usual order or
py-ports Makefile.

Re: NEW: textproc/py-sphinx_guzzle_theme

[Stuart Henderson]

On 2017/10/19 07:19, Björn Ketelaars wrote:
> I agree. Included in the attached port is the diff below...OK?

Yes, OK sthen@ to import.

Re: NEW: textproc/py-sphinx_guzzle_theme

[Björn Ketelaars]

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
> > >  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

Re: NEW: textproc/py-sphinx_guzzle_theme

[Stuart Henderson]

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
> >  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.

> On the other hand py-sphinx_guzzle_theme is similar to py-sphinx_rtd_theme.
> - py-sphinx_rtd_theme has no DEP on py-sphinx
> - py-sphinx has a RDEP on py-sphinx_rdt_theme
> - py-sphinx is the only port I can find which requires py-sphinx_rdt_theme

rtd_theme does *not* have sphinx as an install_requires. and if sphinx itself
depends on it, it cannot.

Re: NEW: textproc/py-sphinx_guzzle_theme

[Remi Locherer]

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
>  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.

On the other hand py-sphinx_guzzle_theme is similar to py-sphinx_rtd_theme.
- py-sphinx_rtd_theme has no DEP on py-sphinx
- py-sphinx has a RDEP on py-sphinx_rdt_theme
- py-sphinx is the only port I can find which requires py-sphinx_rdt_theme

I'm not sure what the correct answer is.

Re: NEW: textproc/py-sphinx_guzzle_theme

[Daniel Jakots]

On Mon, 16 Oct 2017 13:40:27 +0200, Björn Ketelaars
 wrote:

> attached port

In setup.py there is
install_requires=['Sphinx>=1.2b1'],

shouldn't it need a RDEP then?

Cheers,
Daniel

Re: NEW: textproc/py-sphinx_guzzle_theme

[Remi Locherer]

On Mon, Oct 16, 2017 at 01:40:27PM +0200, Björn Ketelaars wrote:
> On Mon 16/10/2017 08:42, Remi Locherer wrote:
> > On Thu, Oct 12, 2017 at 10:10:58PM +0200, Björn Ketelaars wrote:
> > > On Sun 08/10/2017 07:42, Björn Ketelaars wrote:
> > > > Please find enclosed a port for Guzzle Sphinx Theme, which is needed as 
> > > > build
> > > > dependency for the upcoming borgbackup update.
> > > > 
> > > > OK?
> > > > 
> > > 
> > > Ping?
> > 
> > Hi Björn,
> > 
> > I tried to build the package for Guzzle Sphinx Theme but it fails:
> > 
> > mistral ..zzle_theme$ make package
[...]
> > 
> > Remi
> 
> Remi, thank you for your feedback!
> 
> I made a mistake, which is easily fixed as you can see below. Fix is included
> in the attached port.
> 
> OK?

yes, that fixed it.

OK remi@

> 
> 
> --- textproc/py-sphinx_guzzle_theme/pkg/PLIST
> +++ textproc/py-sphinx_guzzle_theme/pkg/PLIST
> @@ -8,7 +8,7 @@ 
> lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme-${MODPY_EGG_VERSION
>  
> lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/requires.txt
>  
> lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/top_level.txt
>  lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/__init__.py
> -lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/${MODPY_PYCACHE}/
> +${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/${MODPY_PYCACHE}/
>  
> lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
>  
> lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/guzzle_sphinx_theme/
>  
> lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/guzzle_sphinx_theme/.DS_Store
> 
> -- 
> Björn Ketelaars
> GPG key: 0x4F0E5F21

Re: NEW: textproc/py-sphinx_guzzle_theme

[Björn Ketelaars]

On Mon 16/10/2017 08:42, Remi Locherer wrote:
> On Thu, Oct 12, 2017 at 10:10:58PM +0200, Björn Ketelaars wrote:
> > On Sun 08/10/2017 07:42, Björn Ketelaars wrote:
> > > Please find enclosed a port for Guzzle Sphinx Theme, which is needed as 
> > > build
> > > dependency for the upcoming borgbackup update.
> > > 
> > > OK?
> > > 
> > 
> > Ping?
> 
> Hi Björn,
> 
> I tried to build the package for Guzzle Sphinx Theme but it fails:
> 
> mistral ..zzle_theme$ make package
> ===>  Looking for py-sphinx_guzzle_theme-0.7.11.tgz in $PKG_PATH - not found
> *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:1935 
> '/usr/ports/packages/amd64/cache/py-sphinx_guzzle_theme-0.7.11.tgz': @if 
> /us...)
> `/usr/ports/pobj/py-sphinx_guzzle_theme-0.7.11/fake-amd64/.fake_done' is up 
> to date.
> ===>  Building package for py-sphinx_guzzle_theme-0.7.11
> Create /usr/ports/packages/amd64/all/py-sphinx_guzzle_theme-0.7.11.tgz
> Error: duplicate item in packing-list 
> /usr/local/lib/python2.7/site-packages/guzzle_sphinx_theme
> Fatal error: can't continue
>  at /usr/libdata/perl5/OpenBSD/PkgCreate.pm line 1514.
> *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:1950 
> '/usr/ports/packages/amd64/all/py-sphinx_guzzle_theme-0.7.11.tgz')
> *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:1943 
> '/usr/ports/packages/amd64/all/py-sphinx_guzzle_theme-0.7.11.tgz')
> *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2446 
> '_internal-package')
> *** Error 1 in /usr/ports/textproc/py-sphinx_guzzle_theme 
> (/usr/ports/infrastructure/mk/bsd.port.mk:2425 'package')
> mistral 1 ..zzle_theme$
> 
> 
> Remi

Remi, thank you for your feedback!

I made a mistake, which is easily fixed as you can see below. Fix is included
in the attached port.

OK?


--- textproc/py-sphinx_guzzle_theme/pkg/PLIST
+++ textproc/py-sphinx_guzzle_theme/pkg/PLIST
@@ -8,7 +8,7 @@ 
lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme-${MODPY_EGG_VERSION
 
lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/requires.txt
 
lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/top_level.txt
 lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/__init__.py
-lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/${MODPY_PYCACHE}/
+${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/${MODPY_PYCACHE}/
 
lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
 
lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/guzzle_sphinx_theme/
 
lib/python${MODPY_VERSION}/site-packages/guzzle_sphinx_theme/guzzle_sphinx_theme/.DS_Store

-- 
Björn Ketelaars
GPG key: 0x4F0E5F21


py-sphinx_guzzle_theme.tgz
Description: application/tar-gz

Re: NEW: textproc/py-sphinx_guzzle_theme

[Remi Locherer]

On Thu, Oct 12, 2017 at 10:10:58PM +0200, Björn Ketelaars wrote:
> On Sun 08/10/2017 07:42, Björn Ketelaars wrote:
> > Please find enclosed a port for Guzzle Sphinx Theme, which is needed as 
> > build
> > dependency for the upcoming borgbackup update.
> > 
> > OK?
> > 
> 
> Ping?

Hi Björn,

I tried to build the package for Guzzle Sphinx Theme but it fails:

mistral ..zzle_theme$ make package
===>  Looking for py-sphinx_guzzle_theme-0.7.11.tgz in $PKG_PATH - not found
*** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:1935 
'/usr/ports/packages/amd64/cache/py-sphinx_guzzle_theme-0.7.11.tgz': @if /us...)
`/usr/ports/pobj/py-sphinx_guzzle_theme-0.7.11/fake-amd64/.fake_done' is up to 
date.
===>  Building package for py-sphinx_guzzle_theme-0.7.11
Create /usr/ports/packages/amd64/all/py-sphinx_guzzle_theme-0.7.11.tgz
Error: duplicate item in packing-list 
/usr/local/lib/python2.7/site-packages/guzzle_sphinx_theme
Fatal error: can't continue
 at /usr/libdata/perl5/OpenBSD/PkgCreate.pm line 1514.
*** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:1950 
'/usr/ports/packages/amd64/all/py-sphinx_guzzle_theme-0.7.11.tgz')
*** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:1943 
'/usr/ports/packages/amd64/all/py-sphinx_guzzle_theme-0.7.11.tgz')
*** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2446 
'_internal-package')
*** Error 1 in /usr/ports/textproc/py-sphinx_guzzle_theme 
(/usr/ports/infrastructure/mk/bsd.port.mk:2425 'package')
mistral 1 ..zzle_theme$


Remi

Re: NEW: textproc/py-sphinx_guzzle_theme

[Björn Ketelaars]

On Sun 08/10/2017 07:42, Björn Ketelaars wrote:
> Please find enclosed a port for Guzzle Sphinx Theme, which is needed as build
> dependency for the upcoming borgbackup update.
> 
> OK?
> 

Ping?

NEW: textproc/py-sphinx_guzzle_theme

[Björn Ketelaars]

Please find enclosed a port for Guzzle Sphinx Theme, which is needed as build
dependency for the upcoming borgbackup update.

OK?

-- 
Björn Ketelaars
GPG key: 0x4F0E5F21


py-sphinx_guzzle_theme.tgz
Description: application/tar-gz