Re: [Update] databases/py-redis to 4.3.4

[Uwe Werler]

On 12 Sep 21:26, Stuart Henderson wrote:
> On 2022/09/12 20:12, Uwe Werler wrote:
> > Hi all,
> > 
> > here's an update for py-redis to 4.3.4 (with cluster support since 4.1.0).
> 
> It needs some new deps, see the install_requires block in setup.py - tgz
> attached for those.
> 
> Here's a tweaked version of the port update diff.
> 
> The redismod/cluster tests are failing for me, they might just need
> more setting up, anyway let's disable them for now so that it's easier
> to be convinced that basic functionality is working ok.

Hi Stuart,

tested the modules (incl. dependencies) now with my fixed redis_cache.py and
redis_return.py on salt 3003.5 and works like expected. I will have a look at
the cluster tests then.

Regards Uwe

> 
> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/databases/py-redis/Makefile,v
> retrieving revision 1.42
> diff -u -p -r1.42 Makefile
> --- Makefile  11 Mar 2022 18:31:41 -  1.42
> +++ Makefile  12 Sep 2022 20:23:41 -
> @@ -1,16 +1,13 @@
>  COMMENT =Python interface to Redis
>  
> -MODPY_EGG_VERSION =  3.5.3
> -
> -GH_ACCOUNT = andymccurdy
> -GH_PROJECT = redis-py
> -GH_TAGNAME = ${MODPY_EGG_VERSION}
> +MODPY_EGG_VERSION =  4.3.4
>  
> +DISTNAME =   redis-${MODPY_EGG_VERSION}
>  PKGNAME =py-redis-${MODPY_EGG_VERSION}
> +
>  CATEGORIES = databases
> -REVISION =   2
>  
> -HOMEPAGE =   https://github.com/andymccurdy/redis-py/
> +HOMEPAGE =   https://github.com/redis/redis-py
>  
>  MAINTAINER = Remi Pointel 
>  
> @@ -18,17 +15,24 @@ MAINTAINER =  Remi Pointel   PERMIT_PACKAGE = Yes
>  
>  MODULES =lang/python
> +MODPY_PEP517 =   setuptools
> +MODPY_PI =   Yes
> +MODPY_PYTEST_ARGS =  --disable-pytest-warnings \
> + -m "onlynoncluster and not redismod" \
> + --ignore tests/test_ssl.py
>  
>  FLAVORS =python3
>  FLAVOR = python3
>  
> -MODPY_SETUPTOOLS =   Yes
> -MODPY_PYTEST =   Yes
> -
>  # Be sure to have a redis server running on localhost
>  TEST_IS_INTERACTIVE =Yes
>  
>  TEST_DEPENDS =   databases/redis \
> - devel/py-mock${MODPY_FLAVOR}
> + devel/py-mock${MODPY_FLAVOR} \
> + devel/py-test-asyncio${MODPY_FLAVOR}
> +
> +RUN_DEPENDS =devel/py-deprecated${MODPY_FLAVOR} \
> + devel/py-async-timeout${MODPY_FLAVOR} \
> + sysutils/py-packaging${MODPY_FLAVOR}
>  
>  .include 
> Index: distinfo
> ===
> RCS file: /cvs/ports/databases/py-redis/distinfo,v
> retrieving revision 1.25
> diff -u -p -r1.25 distinfo
> --- distinfo  6 Oct 2020 20:29:16 -   1.25
> +++ distinfo  12 Sep 2022 20:23:41 -
> @@ -1,2 +1,2 @@
> -SHA256 (redis-py-3.5.3.tar.gz) = Dj7wSvV9FyB8LxOklkGmO5jgBAoQdWqCR+gzRotaggY=
> -SIZE (redis-py-3.5.3.tar.gz) = 129884
> +SHA256 (redis-4.3.4.tar.gz) = 3fJwcd9K3zghxPLKWdZ1JcOoLl8mi+2XuBPLT6v4eIA=
> +SIZE (redis-4.3.4.tar.gz) = 4569389
> Index: pkg/PLIST
> ===
> RCS file: /cvs/ports/databases/py-redis/pkg/PLIST,v
> retrieving revision 1.12
> diff -u -p -r1.12 PLIST
> --- pkg/PLIST 11 Mar 2022 18:31:41 -  1.12
> +++ pkg/PLIST 12 Sep 2022 20:23:41 -
> @@ -1,25 +1,209 @@
>  @pkgpath databases/py-redis
>  lib/python${MODPY_VERSION}/site-packages/redis/
> -lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/
> -lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/PKG-INFO
> -lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/SOURCES.txt
> -lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/dependency_links.txt
> -lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/requires.txt
> -lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/top_level.txt
> +lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/
> +lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/LICENSE
> +lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/METADATA
> +lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/RECORD
> +lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/WHEEL
> +lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/top_level.txt
>  lib/python${MODPY_VERSION}/site-packages/redis/__init__.py
>  
> ${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}/
> 

Re: [Update] databases/py-redis to 4.3.4

[Stuart Henderson]

On 2022/09/12 20:12, Uwe Werler wrote:
> Hi all,
> 
> here's an update for py-redis to 4.3.4 (with cluster support since 4.1.0).

It needs some new deps, see the install_requires block in setup.py - tgz
attached for those.

Here's a tweaked version of the port update diff.

The redismod/cluster tests are failing for me, they might just need
more setting up, anyway let's disable them for now so that it's easier
to be convinced that basic functionality is working ok.


Index: Makefile
===
RCS file: /cvs/ports/databases/py-redis/Makefile,v
retrieving revision 1.42
diff -u -p -r1.42 Makefile
--- Makefile11 Mar 2022 18:31:41 -  1.42
+++ Makefile12 Sep 2022 20:23:41 -
@@ -1,16 +1,13 @@
 COMMENT =  Python interface to Redis
 
-MODPY_EGG_VERSION =3.5.3
-
-GH_ACCOUNT =   andymccurdy
-GH_PROJECT =   redis-py
-GH_TAGNAME =   ${MODPY_EGG_VERSION}
+MODPY_EGG_VERSION =4.3.4
 
+DISTNAME = redis-${MODPY_EGG_VERSION}
 PKGNAME =  py-redis-${MODPY_EGG_VERSION}
+
 CATEGORIES =   databases
-REVISION = 2
 
-HOMEPAGE = https://github.com/andymccurdy/redis-py/
+HOMEPAGE = https://github.com/redis/redis-py
 
 MAINTAINER =   Remi Pointel 
 
@@ -18,17 +15,24 @@ MAINTAINER =Remi Pointel 
Index: distinfo
===
RCS file: /cvs/ports/databases/py-redis/distinfo,v
retrieving revision 1.25
diff -u -p -r1.25 distinfo
--- distinfo6 Oct 2020 20:29:16 -   1.25
+++ distinfo12 Sep 2022 20:23:41 -
@@ -1,2 +1,2 @@
-SHA256 (redis-py-3.5.3.tar.gz) = Dj7wSvV9FyB8LxOklkGmO5jgBAoQdWqCR+gzRotaggY=
-SIZE (redis-py-3.5.3.tar.gz) = 129884
+SHA256 (redis-4.3.4.tar.gz) = 3fJwcd9K3zghxPLKWdZ1JcOoLl8mi+2XuBPLT6v4eIA=
+SIZE (redis-4.3.4.tar.gz) = 4569389
Index: pkg/PLIST
===
RCS file: /cvs/ports/databases/py-redis/pkg/PLIST,v
retrieving revision 1.12
diff -u -p -r1.12 PLIST
--- pkg/PLIST   11 Mar 2022 18:31:41 -  1.12
+++ pkg/PLIST   12 Sep 2022 20:23:41 -
@@ -1,25 +1,209 @@
 @pkgpath databases/py-redis
 lib/python${MODPY_VERSION}/site-packages/redis/
-lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/
-lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/PKG-INFO
-lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/SOURCES.txt
-lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/dependency_links.txt
-lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/requires.txt
-lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/top_level.txt
+lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/
+lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/LICENSE
+lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/METADATA
+lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/RECORD
+lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/WHEEL
+lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}.dist-info/top_level.txt
 lib/python${MODPY_VERSION}/site-packages/redis/__init__.py
 
${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}/
+lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
 
lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}_compat.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}backoff.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
+lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}backoff.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}client.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
 
lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}client.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}cluster.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
+lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}cluster.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}compat.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
+lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}compat.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}connection.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
 

[UPDATE] databases/py-redis

[Remi Pointel]

Hi,

this diff updates py-redis to latest release.

Ok?

Cheers,

Remi.
Index: Makefile
===
RCS file: /cvs/ports/databases/py-redis/Makefile,v
retrieving revision 1.37
diff -u -p -u -p -r1.37 Makefile
--- Makefile3 Jul 2020 21:12:37 -   1.37
+++ Makefile20 Sep 2020 15:49:10 -
@@ -2,7 +2,7 @@
 
 COMMENT =  Python interface to Redis
 
-MODPY_EGG_VERSION =3.4.1
+MODPY_EGG_VERSION =3.5.3
 
 GH_ACCOUNT =   andymccurdy
 GH_PROJECT =   redis-py
@@ -10,7 +10,6 @@ GH_TAGNAME =  ${MODPY_EGG_VERSION}
 
 PKGNAME =  py-redis-${MODPY_EGG_VERSION}
 CATEGORIES =   databases
-REVISION = 0
 
 HOMEPAGE = https://github.com/andymccurdy/redis-py/
 
Index: distinfo
===
RCS file: /cvs/ports/databases/py-redis/distinfo,v
retrieving revision 1.24
diff -u -p -u -p -r1.24 distinfo
--- distinfo16 Feb 2020 13:02:19 -  1.24
+++ distinfo20 Sep 2020 15:49:10 -
@@ -1,2 +1,2 @@
-SHA256 (redis-py-3.4.1.tar.gz) = S2ygO5DMShghLunyMeH8iyEGJWp24w7qdsPXzBEFc60=
-SIZE (redis-py-3.4.1.tar.gz) = 126587
+SHA256 (redis-py-3.5.3.tar.gz) = Dj7wSvV9FyB8LxOklkGmO5jgBAoQdWqCR+gzRotaggY=
+SIZE (redis-py-3.5.3.tar.gz) = 129884

Re: UPDATE: databases/py-redis-3.0.1

[Edward Lopez-Acosta]

ping?

Edward Lopez-Acosta

On 12/6/18 5:39 PM, Edward Lopez-Acosta wrote:

Resubmitting the diff so its easier for commit.

I replaced the py-py test depend with py-mock like it should be. py-py 
is pulled is as a dependency of mock.


Edward Lopez-Acosta

On 12/6/18 4:26 AM, Stuart Henderson wrote:

On 2018/12/05 18:47, Edward Lopez-Acosta wrote:

Backwards incompatible changes in this, but checking sqlports I found
nothing that depends on this. I installed this package and ran a 
series of

tests on it using some of my old scripts and didn't hit issues.

$ sqlite3 /usr/local/share/sqlports "select * from depends where 
dependspath
like 'databases/py-redis' or dependspath like 
'databases/py-redis,python3'"

| wc -l


btw you can do "where dependspath like 'databases/py-redis%'" to cover
both flavours.

Tests also require devel/py-mock:

===>  Regression tests for py-redis-3.0.1
running test
Searching for mock
Reading https://pypi.org/simple/mock/
Download error on https://pypi.org/simple/mock/: [Errno 65] No route 
to host -- Some packages may not be found!

Couldn't find index page for 'mock' (maybe misspelled?)
Scanning index of all packages (this may take a while)
Reading https://pypi.org/simple/
Download error on https://pypi.org/simple/: [Errno 65] No route to 
host -- Some packages may not be found!

No local packages or working download links found for mock
error: Could not find suitable distribution for Requirement.parse('mock')

Otherwise looks good to me.

Re: UPDATE: databases/py-redis-3.0.1

[Edward Lopez-Acosta]

Resubmitting the diff so its easier for commit.

I replaced the py-py test depend with py-mock like it should be. py-py 
is pulled is as a dependency of mock.


Edward Lopez-Acosta

On 12/6/18 4:26 AM, Stuart Henderson wrote:

On 2018/12/05 18:47, Edward Lopez-Acosta wrote:

Backwards incompatible changes in this, but checking sqlports I found
nothing that depends on this. I installed this package and ran a series of
tests on it using some of my old scripts and didn't hit issues.

$ sqlite3 /usr/local/share/sqlports "select * from depends where dependspath
like 'databases/py-redis' or dependspath like 'databases/py-redis,python3'"
| wc -l


btw you can do "where dependspath like 'databases/py-redis%'" to cover
both flavours.

Tests also require devel/py-mock:

===>  Regression tests for py-redis-3.0.1
running test
Searching for mock
Reading https://pypi.org/simple/mock/
Download error on https://pypi.org/simple/mock/: [Errno 65] No route to host -- 
Some packages may not be found!
Couldn't find index page for 'mock' (maybe misspelled?)
Scanning index of all packages (this may take a while)
Reading https://pypi.org/simple/
Download error on https://pypi.org/simple/: [Errno 65] No route to host -- Some 
packages may not be found!
No local packages or working download links found for mock
error: Could not find suitable distribution for Requirement.parse('mock')

Otherwise looks good to me.

diff --git Makefile Makefile
index 0d6e4a7e5b6..d335eed8153 100644
--- Makefile
+++ Makefile
@@ -2,12 +2,11 @@
 
 COMMENT =		Python interface to Redis
 
-MODPY_EGG_VERSION =	2.10.6
+MODPY_EGG_VERSION =	3.0.1
 
 GH_ACCOUNT =		andymccurdy
 GH_PROJECT =		redis-py
 GH_TAGNAME =		${MODPY_EGG_VERSION}
-REVISION =		0
 
 DISTNAME =		${GH_PROJECT}-${MODPY_EGG_VERSION}
 PKGNAME =		py-redis-${MODPY_EGG_VERSION}
@@ -31,7 +30,7 @@ MODPY_SETUPTOOLS =	Yes
 TEST_IS_INTERACTIVE =	Yes
 
 TEST_DEPENDS =		databases/redis \
-			devel/py-py${MODPY_FLAVOR} \
+			devel/py-mock${MODPY_FLAVOR} \
 			devel/py-test${MODPY_FLAVOR}>=2.5.0
 
 .include 
diff --git distinfo distinfo
index f1c40bcf07d..e1591ccb4e3 100644
--- distinfo
+++ distinfo
@@ -1,2 +1,2 @@
-SHA256 (redis-py-2.10.6.tar.gz) = 4o4kU51lkKi4hJjy0w9/4+K+EXHXqR/h+mxbaYJRkBU=
-SIZE (redis-py-2.10.6.tar.gz) = 94869
+SHA256 (redis-py-3.0.1.tar.gz) = 0hAZRs9YKu5y1zvu1egVzNpMitT8Kh1ZORMV+ucFnqM=
+SIZE (redis-py-3.0.1.tar.gz) = 106780
diff --git pkg/PLIST pkg/PLIST
index c73d0d89da2..2252caccca2 100644
--- pkg/PLIST
+++ pkg/PLIST
@@ -4,6 +4,7 @@ lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VE
 lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/PKG-INFO
 lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/SOURCES.txt
 lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/dependency_links.txt
+lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/requires.txt
 lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/top_level.txt
 lib/python${MODPY_VERSION}/site-packages/redis/__init__.py
 ${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}/

Re: UPDATE: databases/py-redis-3.0.1

[Edward Lopez-Acosta]

Hi Stuart,

It does look like the py-py requirement changed to mock as you noted. I didn't 
notice the warning initially, my mistake.

Would you like me to submit a new diff this evening with that change? I see the 
same number of tests pass and be skipped.

Thanks for the sqlite tip as well. I am going to update the application I wrote 
with that in mind, and add clarity to my output.

On December 6, 2018 10:26:07 AM UTC, Stuart Henderson  
wrote:
>On 2018/12/05 18:47, Edward Lopez-Acosta wrote:
>> Backwards incompatible changes in this, but checking sqlports I found
>> nothing that depends on this. I installed this package and ran a
>series of
>> tests on it using some of my old scripts and didn't hit issues.
>> 
>> $ sqlite3 /usr/local/share/sqlports "select * from depends where
>dependspath
>> like 'databases/py-redis' or dependspath like
>'databases/py-redis,python3'"
>> | wc -l
>
>btw you can do "where dependspath like 'databases/py-redis%'" to cover
>both flavours.
>
>Tests also require devel/py-mock:
>
>===>  Regression tests for py-redis-3.0.1
>running test
>Searching for mock
>Reading https://pypi.org/simple/mock/
>Download error on https://pypi.org/simple/mock/: [Errno 65] No route to
>host -- Some packages may not be found!
>Couldn't find index page for 'mock' (maybe misspelled?)
>Scanning index of all packages (this may take a while)
>Reading https://pypi.org/simple/
>Download error on https://pypi.org/simple/: [Errno 65] No route to host
>-- Some packages may not be found!
>No local packages or working download links found for mock
>error: Could not find suitable distribution for
>Requirement.parse('mock')
>
>Otherwise looks good to me.

Re: UPDATE: databases/py-redis-3.0.1

[Stuart Henderson]

On 2018/12/05 18:47, Edward Lopez-Acosta wrote:
> Backwards incompatible changes in this, but checking sqlports I found
> nothing that depends on this. I installed this package and ran a series of
> tests on it using some of my old scripts and didn't hit issues.
> 
> $ sqlite3 /usr/local/share/sqlports "select * from depends where dependspath
> like 'databases/py-redis' or dependspath like 'databases/py-redis,python3'"
> | wc -l

btw you can do "where dependspath like 'databases/py-redis%'" to cover
both flavours.

Tests also require devel/py-mock:

===>  Regression tests for py-redis-3.0.1
running test
Searching for mock
Reading https://pypi.org/simple/mock/
Download error on https://pypi.org/simple/mock/: [Errno 65] No route to host -- 
Some packages may not be found!
Couldn't find index page for 'mock' (maybe misspelled?)
Scanning index of all packages (this may take a while)
Reading https://pypi.org/simple/
Download error on https://pypi.org/simple/: [Errno 65] No route to host -- Some 
packages may not be found!
No local packages or working download links found for mock
error: Could not find suitable distribution for Requirement.parse('mock')

Otherwise looks good to me.

UPDATE: databases/py-redis-3.0.1

[Edward Lopez-Acosta]

Backwards incompatible changes in this, but checking sqlports I found 
nothing that depends on this. I installed this package and ran a series 
of tests on it using some of my old scripts and didn't hit issues.


$ sqlite3 /usr/local/share/sqlports "select * from depends where 
dependspath like 'databases/py-redis' or dependspath like 
'databases/py-redis,python3'" | wc -l

   0

Version bump, no new patches needed, MAINTAINER CC, builds and installs 
fine on amd64.


diff generated with `git diff --relative .` in the port directory for 
easy application with patch.


Test results for py3 below but are the same for py2. This is using Redis 
4 which is currently in the tree.


= test session starts 
==

platform openbsd6 -- Python 3.6.7, pytest-3.5.0, py-1.5.3, pluggy-0.6.0
rootdir: /usr/ports/pobj/py-redis-3.0.1-python3/redis-py-3.0.1, inifile:
collected 384 items

tests/test_commands.py ...sss... 
[ 12%]
 
[ 31%]
.... 
[ 50%]
..s.sss... 
[ 66%]
tests/test_connection_pool.py .. 
[ 77%]
. 
[ 77%]
tests/test_encoding.py .. 
[ 79%]
tests/test_lock.py . 
[ 83%]
tests/test_pipeline.py . 
[ 88%]
tests/test_pubsub.py ... 
[ 95%]
tests/test_scripting.py ... 
[ 96%]
tests/test_sentinel.py  
[100%]


 357 passed, 27 skipped in 9.77 seconds 



Thoughts?
--
Edward Lopez-Acosta
diff --git Makefile Makefile
index 0d6e4a7e5b6..315b9964ca6 100644
--- Makefile
+++ Makefile
@@ -2,12 +2,11 @@
 
 COMMENT =		Python interface to Redis
 
-MODPY_EGG_VERSION =	2.10.6
+MODPY_EGG_VERSION =	3.0.1
 
 GH_ACCOUNT =		andymccurdy
 GH_PROJECT =		redis-py
 GH_TAGNAME =		${MODPY_EGG_VERSION}
-REVISION =		0
 
 DISTNAME =		${GH_PROJECT}-${MODPY_EGG_VERSION}
 PKGNAME =		py-redis-${MODPY_EGG_VERSION}
diff --git distinfo distinfo
index f1c40bcf07d..e1591ccb4e3 100644
--- distinfo
+++ distinfo
@@ -1,2 +1,2 @@
-SHA256 (redis-py-2.10.6.tar.gz) = 4o4kU51lkKi4hJjy0w9/4+K+EXHXqR/h+mxbaYJRkBU=
-SIZE (redis-py-2.10.6.tar.gz) = 94869
+SHA256 (redis-py-3.0.1.tar.gz) = 0hAZRs9YKu5y1zvu1egVzNpMitT8Kh1ZORMV+ucFnqM=
+SIZE (redis-py-3.0.1.tar.gz) = 106780
diff --git pkg/PLIST pkg/PLIST
index c73d0d89da2..2252caccca2 100644
--- pkg/PLIST
+++ pkg/PLIST
@@ -4,6 +4,7 @@ lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VE
 lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/PKG-INFO
 lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/SOURCES.txt
 lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/dependency_links.txt
+lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/requires.txt
 lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/top_level.txt
 lib/python${MODPY_VERSION}/site-packages/redis/__init__.py
 ${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/redis/${MODPY_PYCACHE}/

[UPDATE] databases/py-redis

[Remi Pointel]

Hi,

this is the diff to update py-redis to latest release.

Ok?

Cheers,

Remi.Index: Makefile
===
RCS file: /cvs/ports/databases/py-redis/Makefile,v
retrieving revision 1.31
diff -u -p -u -p -r1.31 Makefile
--- Makefile	15 Nov 2015 15:28:26 -	1.31
+++ Makefile	3 Nov 2017 16:07:26 -
@@ -2,7 +2,7 @@
 
 COMMENT =		Python interface to Redis
 
-MODPY_EGG_VERSION =	2.10.5
+MODPY_EGG_VERSION =	2.10.6
 
 GH_ACCOUNT =		andymccurdy
 GH_PROJECT =		redis-py
Index: distinfo
===
RCS file: /cvs/ports/databases/py-redis/distinfo,v
retrieving revision 1.22
diff -u -p -u -p -r1.22 distinfo
--- distinfo	15 Nov 2015 15:28:26 -	1.22
+++ distinfo	3 Nov 2017 16:07:26 -
@@ -1,2 +1,2 @@
-SHA256 (redis-py-2.10.5.tar.gz) = FZg1rF8JgiuzxXi9+Ei50TemfC5K5LeqBTkNrDMIfe8=
-SIZE (redis-py-2.10.5.tar.gz) = 86029
+SHA256 (redis-py-2.10.6.tar.gz) = 4o4kU51lkKi4hJjy0w9/4+K+EXHXqR/h+mxbaYJRkBU=
+SIZE (redis-py-2.10.6.tar.gz) = 94869

Re: [UPDATE] databases/py-redis

[David Coppa]

Il 15/nov/2015 16:28, "Remi Pointel"  ha scritto:
>
> Hi,
>
> attached is the diff to update py-redis to latest release.
>
> Ok?

Ok dcoppa@

> Cheers,
>
> Remi.

Ciao!
David

[UPDATE] databases/py-redis

[Remi Pointel]

Hi,

attached is the diff to update py-redis to latest release.

Ok?

Cheers,

Remi.
Index: Makefile
===
RCS file: /cvs/ports/databases/py-redis/Makefile,v
retrieving revision 1.30
diff -u -p -u -p -r1.30 Makefile
--- Makefile29 Sep 2015 10:51:14 -  1.30
+++ Makefile15 Nov 2015 15:24:10 -
@@ -2,7 +2,7 @@
 
 COMMENT =  Python interface to Redis
 
-MODPY_EGG_VERSION =2.10.3
+MODPY_EGG_VERSION =2.10.5
 
 GH_ACCOUNT =   andymccurdy
 GH_PROJECT =   redis-py
@@ -11,7 +11,6 @@ GH_TAGNAME =  ${MODPY_EGG_VERSION}
 DISTNAME = ${GH_PROJECT}-${MODPY_EGG_VERSION}
 PKGNAME =  py-redis-${MODPY_EGG_VERSION}
 CATEGORIES =   databases
-REVISION = 0
 
 HOMEPAGE = https://github.com/andymccurdy/redis-py/
 
Index: distinfo
===
RCS file: /cvs/ports/databases/py-redis/distinfo,v
retrieving revision 1.21
diff -u -p -u -p -r1.21 distinfo
--- distinfo16 Sep 2014 07:14:14 -  1.21
+++ distinfo15 Nov 2015 15:24:10 -
@@ -1,2 +1,2 @@
-SHA256 (redis-py-2.10.3.tar.gz) = eu+4Cowu3KVmuNJGXIMscOxh/MfPPPcJUb+dL99S+zY=
-SIZE (redis-py-2.10.3.tar.gz) = 84039
+SHA256 (redis-py-2.10.5.tar.gz) = FZg1rF8JgiuzxXi9+Ei50TemfC5K5LeqBTkNrDMIfe8=
+SIZE (redis-py-2.10.5.tar.gz) = 86029

Re: MODPY_BADEGGS revised (Was: [UPDATE] databases/py-redis)

[David Coppa]

On Thu, 26 Sep 2013, David Coppa wrote:

 On Thu, Sep 26, 2013 at 2:30 PM, Remi Pointel remi.poin...@xiri.fr wrote:
  On Thu, 26 Sep 2013 14:19:28 +0200
  David Coppa dco...@gmail.com wrote:
  On Thu, 26 Sep 2013, David Coppa wrote:
 
  
   Why not getting rid of MODPY_BADEGGS and doing this instead:
  
   This will assure the egg-info directory is always created with the
   right permissions...
  
   Comments?
 
  Oops!
  Now with everything in its right place:
 
  Index: python.port.mk
  ===
  RCS file: /cvs/ports/lang/python/python.port.mk,v
  retrieving revision 1.72
  diff -u -p -u -p -r1.72 python.port.mk
  --- python.port.mk24 Sep 2013 16:55:02 -  1.72
  +++ python.port.mk26 Sep 2013 12:19:08 -
  @@ -91,6 +91,7 @@ MODPY_SETUPUTILS =  Yes
   # The setuptools uses test target
   TEST_TARGET ?=   test
   _MODPY_USERBASE =
  +_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info
   .else
   # Try to detect the case where a port will build regardless of setuptools
   # but the final plist will be different if it's present.
  @@ -117,13 +118,6 @@ MODPY_BIN =  ${LOCALBASE}/bin/python${MO
   MODPY_INCDIR =   
  ${LOCALBASE}/include/python${MODPY_VERSION}${MODPY_LIB_SUFFIX}
   MODPY_LIBDIR =   ${LOCALBASE}/lib/python${MODPY_VERSION}
   MODPY_SITEPKG =  ${MODPY_LIBDIR}/site-packages
  -
  -.if defined(MODPY_BADEGGS)
  -.  for egg in ${MODPY_BADEGGS}
  -_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
  -.  endfor
  -.endif
  -
 
   # usually setup.py but Setup.py can be found too
   MODPY_SETUP ?=   setup.py
 
  Could be useful to test this diff in a bulk, no?
 
 Running against all the ports using py-setuptools should be sufficient.
 
  Remi.

As seen on icb, not all the setuptools based stuff support this
command. Some ports fail with: 

error: invalid command 'egg_info'

New revision of the diff, hope I got it right this time:

Index: python.port.mk
===
RCS file: /cvs/ports/lang/python/python.port.mk,v
retrieving revision 1.72
diff -u -p -u -p -r1.72 python.port.mk
--- python.port.mk  24 Sep 2013 16:55:02 -  1.72
+++ python.port.mk  3 Oct 2013 15:01:04 -
@@ -91,6 +91,7 @@ MODPY_SETUPUTILS =Yes
 # The setuptools uses test target
 TEST_TARGET ?= test
 _MODPY_USERBASE =
+_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} --help-commands | grep -qw egg_info  
${MODPY_CMD} egg_info
 .else
 # Try to detect the case where a port will build regardless of setuptools
 # but the final plist will be different if it's present.
@@ -117,13 +118,6 @@ MODPY_BIN =${LOCALBASE}/bin/python${MO
 MODPY_INCDIR = 
${LOCALBASE}/include/python${MODPY_VERSION}${MODPY_LIB_SUFFIX}
 MODPY_LIBDIR = ${LOCALBASE}/lib/python${MODPY_VERSION}
 MODPY_SITEPKG =${MODPY_LIBDIR}/site-packages
-
-.if defined(MODPY_BADEGGS)
-.  for egg in ${MODPY_BADEGGS}
-_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
-.  endfor
-.endif
-
 
 # usually setup.py but Setup.py can be found too
 MODPY_SETUP ?= setup.py

Re: MODPY_BADEGGS revised (Was: [UPDATE] databases/py-redis)

[Landry Breuil]

On Thu, Oct 03, 2013 at 05:01:33PM +0200, David Coppa wrote:
 On Thu, 26 Sep 2013, David Coppa wrote:
 
  On Thu, Sep 26, 2013 at 2:30 PM, Remi Pointel remi.poin...@xiri.fr wrote:
   On Thu, 26 Sep 2013 14:19:28 +0200
   David Coppa dco...@gmail.com wrote:
   On Thu, 26 Sep 2013, David Coppa wrote:
  
   
Why not getting rid of MODPY_BADEGGS and doing this instead:
   
This will assure the egg-info directory is always created with the
right permissions...
   
Comments?
  
   Oops!
   Now with everything in its right place:
  
   Index: python.port.mk
   ===
   RCS file: /cvs/ports/lang/python/python.port.mk,v
   retrieving revision 1.72
   diff -u -p -u -p -r1.72 python.port.mk
   --- python.port.mk24 Sep 2013 16:55:02 -  1.72
   +++ python.port.mk26 Sep 2013 12:19:08 -
   @@ -91,6 +91,7 @@ MODPY_SETUPUTILS =  Yes
# The setuptools uses test target
TEST_TARGET ?=   test
_MODPY_USERBASE =
   +_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info
.else
# Try to detect the case where a port will build regardless of 
   setuptools
# but the final plist will be different if it's present.
   @@ -117,13 +118,6 @@ MODPY_BIN =  ${LOCALBASE}/bin/python${MO
MODPY_INCDIR =   
   ${LOCALBASE}/include/python${MODPY_VERSION}${MODPY_LIB_SUFFIX}
MODPY_LIBDIR =   ${LOCALBASE}/lib/python${MODPY_VERSION}
MODPY_SITEPKG =  ${MODPY_LIBDIR}/site-packages
   -
   -.if defined(MODPY_BADEGGS)
   -.  for egg in ${MODPY_BADEGGS}
   -_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
   -.  endfor
   -.endif
   -
  
# usually setup.py but Setup.py can be found too
MODPY_SETUP ?=   setup.py
  
   Could be useful to test this diff in a bulk, no?
  
  Running against all the ports using py-setuptools should be sufficient.
  
   Remi.
 
 As seen on icb, not all the setuptools based stuff support this
 command. Some ports fail with: 
 
 error: invalid command 'egg_info'
 
 New revision of the diff, hope I got it right this time:
 
 Index: python.port.mk
 ===
 RCS file: /cvs/ports/lang/python/python.port.mk,v
 retrieving revision 1.72
 diff -u -p -u -p -r1.72 python.port.mk
 --- python.port.mk24 Sep 2013 16:55:02 -  1.72
 +++ python.port.mk3 Oct 2013 15:01:04 -
 @@ -91,6 +91,7 @@ MODPY_SETUPUTILS =  Yes
  # The setuptools uses test target
  TEST_TARGET ?=   test
  _MODPY_USERBASE =
 +_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} --help-commands | grep -qw egg_info 
  ${MODPY_CMD} egg_info
  .else
  # Try to detect the case where a port will build regardless of setuptools
  # but the final plist will be different if it's present.
 @@ -117,13 +118,6 @@ MODPY_BIN =  ${LOCALBASE}/bin/python${MO
  MODPY_INCDIR =   
 ${LOCALBASE}/include/python${MODPY_VERSION}${MODPY_LIB_SUFFIX}
  MODPY_LIBDIR =   ${LOCALBASE}/lib/python${MODPY_VERSION}
  MODPY_SITEPKG =  ${MODPY_LIBDIR}/site-packages
 -
 -.if defined(MODPY_BADEGGS)
 -.  for egg in ${MODPY_BADEGGS}
 -_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
 -.  endfor
 -.endif
 -
  
  # usually setup.py but Setup.py can be found too
  MODPY_SETUP ?=   setup.py
 

That works for the three ports broken by the previous diff (terminator,
py-cef and py-greenlet) but that looks like a huge hack to me.

Landry

Re: MODPY_BADEGGS revised (Was: [UPDATE] databases/py-redis)

[David Coppa]

On Thu, 03 Oct 2013, Landry Breuil wrote:

 That works for the three ports broken by the previous diff (terminator,
 py-cef and py-greenlet) but that looks like a huge hack to me.

We settled on this:

Index: python.port.mk
===
RCS file: /cvs/ports/lang/python/python.port.mk,v
retrieving revision 1.72
diff -u -p -u -p -r1.72 python.port.mk
--- python.port.mk  24 Sep 2013 16:55:02 -  1.72
+++ python.port.mk  3 Oct 2013 16:06:24 -
@@ -91,6 +91,7 @@ MODPY_SETUPUTILS =Yes
 # The setuptools uses test target
 TEST_TARGET ?= test
 _MODPY_USERBASE =
+_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info || true
 .else
 # Try to detect the case where a port will build regardless of setuptools
 # but the final plist will be different if it's present.
@@ -117,13 +118,6 @@ MODPY_BIN =${LOCALBASE}/bin/python${MO
 MODPY_INCDIR = 
${LOCALBASE}/include/python${MODPY_VERSION}${MODPY_LIB_SUFFIX}
 MODPY_LIBDIR = ${LOCALBASE}/lib/python${MODPY_VERSION}
 MODPY_SITEPKG =${MODPY_LIBDIR}/site-packages
-
-.if defined(MODPY_BADEGGS)
-.  for egg in ${MODPY_BADEGGS}
-_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
-.  endfor
-.endif
-
 
 # usually setup.py but Setup.py can be found too
 MODPY_SETUP ?= setup.py

[UPDATE] databases/py-redis

[Remi Pointel]

Hi,

this is the diff to update to latest release of py-redis: 2.8.0.
You need to have py-test version 2.3.5 to run the test suite.

Ok?

Cheers,

Remi.
Index: Makefile
===
RCS file: /cvs/ports/databases/py-redis/Makefile,v
retrieving revision 1.22
diff -u -p -r1.22 Makefile
--- Makefile18 Jun 2013 11:47:06 -  1.22
+++ Makefile26 Sep 2013 09:04:48 -
@@ -2,7 +2,7 @@
 
 COMMENT =  Python interface to Redis
 
-MODPY_EGG_VERSION =2.7.6
+MODPY_EGG_VERSION =2.8.0
 DISTNAME = redis-py-${MODPY_EGG_VERSION}
 PKGNAME =  py-redis-${MODPY_EGG_VERSION}
 CATEGORIES =   databases
@@ -19,9 +19,7 @@ MODULES = lang/python
 
 MODPY_SETUPTOOLS = Yes
 
-MODPY_BADEGGS =redis
-
-# XXX: test suite requires newer devel/py-test and devel/py-py
-NO_TEST =  Yes
+TEST_DEPENDS = devel/py-test \
+   devel/py-py
 
 .include bsd.port.mk
Index: distinfo
===
RCS file: /cvs/ports/databases/py-redis/distinfo,v
retrieving revision 1.18
diff -u -p -r1.18 distinfo
--- distinfo18 Jun 2013 11:47:06 -  1.18
+++ distinfo26 Sep 2013 09:04:48 -
@@ -1,2 +1,2 @@
-SHA256 (redis-py-2.7.6.tar.gz) = 1cylSlWtSuXS+D80nJf9wMwY5N9AP3jstbFPzWQG4CI=
-SIZE (redis-py-2.7.6.tar.gz) = 53494
+SHA256 (redis-py-2.8.0.tar.gz) = VVYzCysqYWz8k3mutVn7WWTpZESbLJVq39BVPOpknoY=
+SIZE (redis-py-2.8.0.tar.gz) = 54218

Re: [UPDATE] databases/py-redis

[David Coppa]

On Thu, 26 Sep 2013, Remi Pointel wrote:

 Hi,
 
 this is the diff to update to latest release of py-redis: 2.8.0.
 You need to have py-test version 2.3.5 to run the test suite.
 
 Ok?

This has a problem with bad eggs I cannot find an easy solution for...

/usr/ports/databases/py-redis$ make clean fake

...

/usr/ports/databases/py-redis$ make test
=== py-redis-2.8.0 depends on: py-test-* - py-test-2.3.5
=== py-redis-2.8.0 depends on: py-py-* - py-py-1.4.15p0
===  Regression tests for py-redis-2.8.0
running test
running egg_info
writing redis.egg-info/PKG-INFO
error: redis.egg-info/PKG-INFO: Permission denied
*** Error 1 in . (/usr/ports/lang/python/python.port.mk:189 'do-test': @cd 
/usr/ports/pobj/py-redis-2.8.0/redis-py-2.8.0  /usr/bin/env -i ...)
*** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2682 
'/usr/ports/pobj/py-redis-2.8.0/.test_done')
*** Error 1 in /usr/ports/databases/py-redis 
(/usr/ports/infrastructure/mk/bsd.port.mk:2383 'test')

Re: [UPDATE] databases/py-redis

[Remi Pointel]

On Thu, 26 Sep 2013 13:33:42 +0200
David Coppa dco...@gmail.com wrote:

 On Thu, 26 Sep 2013, Remi Pointel wrote:
 
  Hi,
  
  this is the diff to update to latest release of py-redis: 2.8.0.
  You need to have py-test version 2.3.5 to run the test suite.
  
  Ok?
 
 This has a problem with bad eggs I cannot find an easy solution for...
 
 /usr/ports/databases/py-redis$ make clean fake
 
 ...
 
 /usr/ports/databases/py-redis$ make test
 === py-redis-2.8.0 depends on: py-test-* - py-test-2.3.5
 === py-redis-2.8.0 depends on: py-py-* - py-py-1.4.15p0
 ===  Regression tests for py-redis-2.8.0
 running test
 running egg_info
 writing redis.egg-info/PKG-INFO
 error: redis.egg-info/PKG-INFO: Permission denied
 *** Error 1 in . (/usr/ports/lang/python/python.port.mk:189 'do-test': @cd 
 /usr/ports/pobj/py-redis-2.8.0/redis-py-2.8.0  /usr/bin/env -i ...)
 *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2682 
 '/usr/ports/pobj/py-redis-2.8.0/.test_done')
 *** Error 1 in /usr/ports/databases/py-redis 
 (/usr/ports/infrastructure/mk/bsd.port.mk:2383 'test')
 
 

Yes, I had a problem with the bad egg this is why I removed this, and it worked 
for me.
I will retest.

Remi.

Re: [UPDATE] databases/py-redis

[David Coppa]

On Thu, Sep 26, 2013 at 1:36 PM, Remi Pointel remi.poin...@xiri.fr wrote:

 Yes, I had a problem with the bad egg this is why I removed this, and it 
 worked for me.
 I will retest.

If you remove MODPY_BADEGGS = redis, 'make clean' will also fail.

MODPY_BADEGGS revised (Was: [UPDATE] databases/py-redis)

[David Coppa]

Why not getting rid of MODPY_BADEGGS and doing this instead:

---8---

Index: python.port.mk
===
RCS file: /cvs/ports/lang/python/python.port.mk,v
retrieving revision 1.72
diff -u -p -u -p -r1.72 python.port.mk
--- python.port.mk  24 Sep 2013 16:55:02 -  1.72
+++ python.port.mk  26 Sep 2013 11:55:58 -
@@ -118,12 +118,7 @@ MODPY_INCDIR = ${LOCALBASE}/include/pyt
 MODPY_LIBDIR = ${LOCALBASE}/lib/python${MODPY_VERSION}
 MODPY_SITEPKG =${MODPY_LIBDIR}/site-packages
 
-.if defined(MODPY_BADEGGS)
-.  for egg in ${MODPY_BADEGGS}
-_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
-.  endfor
-.endif
-
+_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info
 
 # usually setup.py but Setup.py can be found too
 MODPY_SETUP ?= setup.py

---8---

This will assure the egg-info directory is always created with the
right permissions...

Comments?

Re: MODPY_BADEGGS revised (Was: [UPDATE] databases/py-redis)

[David Coppa]

On Thu, 26 Sep 2013, David Coppa wrote:

 
 Why not getting rid of MODPY_BADEGGS and doing this instead:
 
 ---8---
 
 Index: python.port.mk
 ===
 RCS file: /cvs/ports/lang/python/python.port.mk,v
 retrieving revision 1.72
 diff -u -p -u -p -r1.72 python.port.mk
 --- python.port.mk24 Sep 2013 16:55:02 -  1.72
 +++ python.port.mk26 Sep 2013 11:55:58 -
 @@ -118,12 +118,7 @@ MODPY_INCDIR =   ${LOCALBASE}/include/pyt
  MODPY_LIBDIR =   ${LOCALBASE}/lib/python${MODPY_VERSION}
  MODPY_SITEPKG =  ${MODPY_LIBDIR}/site-packages
  
 -.if defined(MODPY_BADEGGS)
 -.  for egg in ${MODPY_BADEGGS}
 -_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
 -.  endfor
 -.endif
 -
 +_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info
  
  # usually setup.py but Setup.py can be found too
  MODPY_SETUP ?=   setup.py
 
 ---8---
 
 This will assure the egg-info directory is always created with the
 right permissions...
 
 Comments?

Oops!
Now with everything in its right place:

Index: python.port.mk
===
RCS file: /cvs/ports/lang/python/python.port.mk,v
retrieving revision 1.72
diff -u -p -u -p -r1.72 python.port.mk
--- python.port.mk  24 Sep 2013 16:55:02 -  1.72
+++ python.port.mk  26 Sep 2013 12:19:08 -
@@ -91,6 +91,7 @@ MODPY_SETUPUTILS =Yes
 # The setuptools uses test target
 TEST_TARGET ?= test
 _MODPY_USERBASE =
+_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info
 .else
 # Try to detect the case where a port will build regardless of setuptools
 # but the final plist will be different if it's present.
@@ -117,13 +118,6 @@ MODPY_BIN =${LOCALBASE}/bin/python${MO
 MODPY_INCDIR = 
${LOCALBASE}/include/python${MODPY_VERSION}${MODPY_LIB_SUFFIX}
 MODPY_LIBDIR = ${LOCALBASE}/lib/python${MODPY_VERSION}
 MODPY_SITEPKG =${MODPY_LIBDIR}/site-packages
-
-.if defined(MODPY_BADEGGS)
-.  for egg in ${MODPY_BADEGGS}
-_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
-.  endfor
-.endif
-
 
 # usually setup.py but Setup.py can be found too
 MODPY_SETUP ?= setup.py

Re: MODPY_BADEGGS revised (Was: [UPDATE] databases/py-redis)

[Remi Pointel]

On Thu, 26 Sep 2013 14:19:28 +0200
David Coppa dco...@gmail.com wrote:
 On Thu, 26 Sep 2013, David Coppa wrote:
 
  
  Why not getting rid of MODPY_BADEGGS and doing this instead:
  
  ---8---
  
  Index: python.port.mk
  ===
  RCS file: /cvs/ports/lang/python/python.port.mk,v
  retrieving revision 1.72
  diff -u -p -u -p -r1.72 python.port.mk
  --- python.port.mk  24 Sep 2013 16:55:02 -  1.72
  +++ python.port.mk  26 Sep 2013 11:55:58 -
  @@ -118,12 +118,7 @@ MODPY_INCDIR = ${LOCALBASE}/include/pyt
   MODPY_LIBDIR = ${LOCALBASE}/lib/python${MODPY_VERSION}
   MODPY_SITEPKG =${MODPY_LIBDIR}/site-packages
   
  -.if defined(MODPY_BADEGGS)
  -.  for egg in ${MODPY_BADEGGS}
  -_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
  -.  endfor
  -.endif
  -
  +_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info
   
   # usually setup.py but Setup.py can be found too
   MODPY_SETUP ?= setup.py
  
  ---8---
  
  This will assure the egg-info directory is always created with the
  right permissions...
  
  Comments?
 
 Oops!
 Now with everything in its right place:
 
 Index: python.port.mk
 ===
 RCS file: /cvs/ports/lang/python/python.port.mk,v
 retrieving revision 1.72
 diff -u -p -u -p -r1.72 python.port.mk
 --- python.port.mk24 Sep 2013 16:55:02 -  1.72
 +++ python.port.mk26 Sep 2013 12:19:08 -
 @@ -91,6 +91,7 @@ MODPY_SETUPUTILS =  Yes
  # The setuptools uses test target
  TEST_TARGET ?=   test
  _MODPY_USERBASE =
 +_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info
  .else
  # Try to detect the case where a port will build regardless of setuptools
  # but the final plist will be different if it's present.
 @@ -117,13 +118,6 @@ MODPY_BIN =  ${LOCALBASE}/bin/python${MO
  MODPY_INCDIR =   
 ${LOCALBASE}/include/python${MODPY_VERSION}${MODPY_LIB_SUFFIX}
  MODPY_LIBDIR =   ${LOCALBASE}/lib/python${MODPY_VERSION}
  MODPY_SITEPKG =  ${MODPY_LIBDIR}/site-packages
 -
 -.if defined(MODPY_BADEGGS)
 -.  for egg in ${MODPY_BADEGGS}
 -_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
 -.  endfor
 -.endif
 -
  
  # usually setup.py but Setup.py can be found too
  MODPY_SETUP ?=   setup.py

Could be useful to test this diff in a bulk, no?

Remi.

Re: MODPY_BADEGGS revised (Was: [UPDATE] databases/py-redis)

[David Coppa]

On Thu, Sep 26, 2013 at 2:30 PM, Remi Pointel remi.poin...@xiri.fr wrote:
 On Thu, 26 Sep 2013 14:19:28 +0200
 David Coppa dco...@gmail.com wrote:
 On Thu, 26 Sep 2013, David Coppa wrote:

 
  Why not getting rid of MODPY_BADEGGS and doing this instead:
 
  ---8---
 
  Index: python.port.mk
  ===
  RCS file: /cvs/ports/lang/python/python.port.mk,v
  retrieving revision 1.72
  diff -u -p -u -p -r1.72 python.port.mk
  --- python.port.mk  24 Sep 2013 16:55:02 -  1.72
  +++ python.port.mk  26 Sep 2013 11:55:58 -
  @@ -118,12 +118,7 @@ MODPY_INCDIR = ${LOCALBASE}/include/pyt
   MODPY_LIBDIR = ${LOCALBASE}/lib/python${MODPY_VERSION}
   MODPY_SITEPKG =${MODPY_LIBDIR}/site-packages
 
  -.if defined(MODPY_BADEGGS)
  -.  for egg in ${MODPY_BADEGGS}
  -_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
  -.  endfor
  -.endif
  -
  +_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info
 
   # usually setup.py but Setup.py can be found too
   MODPY_SETUP ?= setup.py
 
  ---8---
 
  This will assure the egg-info directory is always created with the
  right permissions...
 
  Comments?

 Oops!
 Now with everything in its right place:

 Index: python.port.mk
 ===
 RCS file: /cvs/ports/lang/python/python.port.mk,v
 retrieving revision 1.72
 diff -u -p -u -p -r1.72 python.port.mk
 --- python.port.mk24 Sep 2013 16:55:02 -  1.72
 +++ python.port.mk26 Sep 2013 12:19:08 -
 @@ -91,6 +91,7 @@ MODPY_SETUPUTILS =  Yes
  # The setuptools uses test target
  TEST_TARGET ?=   test
  _MODPY_USERBASE =
 +_MODPY_PRE_BUILD_STEPS += ;${MODPY_CMD} egg_info
  .else
  # Try to detect the case where a port will build regardless of setuptools
  # but the final plist will be different if it's present.
 @@ -117,13 +118,6 @@ MODPY_BIN =  ${LOCALBASE}/bin/python${MO
  MODPY_INCDIR =   
 ${LOCALBASE}/include/python${MODPY_VERSION}${MODPY_LIB_SUFFIX}
  MODPY_LIBDIR =   ${LOCALBASE}/lib/python${MODPY_VERSION}
  MODPY_SITEPKG =  ${MODPY_LIBDIR}/site-packages
 -
 -.if defined(MODPY_BADEGGS)
 -.  for egg in ${MODPY_BADEGGS}
 -_MODPY_PRE_BUILD_STEPS += ;mkdir -p ${WRKBUILD}/${egg}.egg-info
 -.  endfor
 -.endif
 -

  # usually setup.py but Setup.py can be found too
  MODPY_SETUP ?=   setup.py

 Could be useful to test this diff in a bulk, no?

Running against all the ports using py-setuptools should be sufficient.

 Remi.

ciao,
David

[UPDATE] databases/py-redis

[Remi Pointel]

Hi,

this is the diff to update py-redis to 2.4.9.

Is it ok ?

Cheers,

Remi.
Index: Makefile
===
RCS file: /cvs/ports/databases/py-redis/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile18 Jul 2011 07:57:32 -  1.9
+++ Makefile27 Jul 2011 06:30:58 -
@@ -2,7 +2,7 @@
 
 COMMENT =  Python interface to Redis
 
-MODPY_EGG_VERSION =2.4.7
+MODPY_EGG_VERSION =2.4.9
 DISTNAME = redis-${MODPY_EGG_VERSION}
 PKGNAME =  py-redis-${MODPY_EGG_VERSION}
 CATEGORIES =   databases
Index: distinfo
===
RCS file: /cvs/ports/databases/py-redis/distinfo,v
retrieving revision 1.8
diff -u -p -r1.8 distinfo
--- distinfo18 Jul 2011 07:57:32 -  1.8
+++ distinfo27 Jul 2011 06:30:58 -
@@ -1,5 +1,5 @@
-MD5 (py-redis/redis-2.4.7.tar.gz) = loTlk3lnHE8jk6HU/Abhow==
-RMD160 (py-redis/redis-2.4.7.tar.gz) = groKbY4wtEhDhlxT5fHVEe4hfsg=
-SHA1 (py-redis/redis-2.4.7.tar.gz) = 8jErQkPMXx0L5NPXLo8cm5493aM=
-SHA256 (py-redis/redis-2.4.7.tar.gz) = 
WJFpdtjc+jYD1J2gyYQgfgQZfB1g4oxZ1AwxkvALPP4=
-SIZE (py-redis/redis-2.4.7.tar.gz) = 26281
+MD5 (py-redis/redis-2.4.9.tar.gz) = tRL/N9BsaBPwSlf2RIoeVQ==
+RMD160 (py-redis/redis-2.4.9.tar.gz) = 9wm2uWEW1C86hjgCUyb47rRv7/s=
+SHA1 (py-redis/redis-2.4.9.tar.gz) = N4x4t4dgR9dUBmZXAVQr+MV0pEA=
+SHA256 (py-redis/redis-2.4.9.tar.gz) = 
9OqFdn4DfRqkcScoQNsq9RL+hgQzJGIfiD3nyczr+vE=
+SIZE (py-redis/redis-2.4.9.tar.gz) = 26373

Re: [UPDATE] databases/py-redis

[David Coppa]

On Wed, Jul 27, 2011 at 8:33 AM, Remi Pointel remi.poin...@xiri.fr wrote:
 Hi,

 this is the diff to update py-redis to 2.4.9.

 Is it ok ?

Sure. Go ahead.

ciao,
david

[UPDATE] databases/py-redis

[Remi Pointel]

Hi,

this is the diff to update py-redis to 2.4.6.

Are you ok ?

Cheers,

Remi.
Index: Makefile
===
RCS file: /cvs/ports/databases/py-redis/Makefile,v
retrieving revision 1.7
diff -u -p -r1.7 Makefile
--- Makefile10 Jun 2011 21:18:17 -  1.7
+++ Makefile17 Jul 2011 11:43:34 -
@@ -2,7 +2,7 @@
 
 COMMENT =  Python interface to Redis
 
-MODPY_EGG_VERSION =2.4.5
+MODPY_EGG_VERSION =2.4.6
 DISTNAME = redis-${MODPY_EGG_VERSION}
 PKGNAME =  py-redis-${MODPY_EGG_VERSION}
 CATEGORIES =   databases
Index: distinfo
===
RCS file: /cvs/ports/databases/py-redis/distinfo,v
retrieving revision 1.6
diff -u -p -r1.6 distinfo
--- distinfo10 Jun 2011 21:18:17 -  1.6
+++ distinfo17 Jul 2011 11:43:34 -
@@ -1,5 +1,5 @@
-MD5 (py-redis/redis-2.4.5.tar.gz) = RSNV/t1JC25qZYWXogcKYQ==
-RMD160 (py-redis/redis-2.4.5.tar.gz) = 8+GPHpUtLXHvQ5vD4owWsdPziQA=
-SHA1 (py-redis/redis-2.4.5.tar.gz) = sGHTRjpDRIXp0nsfv8I58tpcAUo=
-SHA256 (py-redis/redis-2.4.5.tar.gz) = 
AIbeHjAzMi5bZEgK1nyqb/xvf61IgVM0WBW4tQLKYm4=
-SIZE (py-redis/redis-2.4.5.tar.gz) = 22017
+MD5 (py-redis/redis-2.4.6.tar.gz) = BgnssuCbINNNlmYxUdIviw==
+RMD160 (py-redis/redis-2.4.6.tar.gz) = P1rEFrbARGNEAGPlXKHa1MlzFzU=
+SHA1 (py-redis/redis-2.4.6.tar.gz) = uCCm6I0yjBZ/4c/AvIBzBocHdq8=
+SHA256 (py-redis/redis-2.4.6.tar.gz) = 
sBi3RUF1toNHPx3+1vPz2dUwqoARN0v9WeBzEbwvSdA=
+SIZE (py-redis/redis-2.4.6.tar.gz) = 24362
Index: pkg/PLIST
===
RCS file: /cvs/ports/databases/py-redis/pkg/PLIST,v
retrieving revision 1.3
diff -u -p -r1.3 PLIST
--- pkg/PLIST   2 Jun 2011 16:02:33 -   1.3
+++ pkg/PLIST   17 Jul 2011 11:43:34 -
@@ -13,5 +13,3 @@ lib/python${MODPY_VERSION}/site-packages
 lib/python${MODPY_VERSION}/site-packages/redis/connection.pyc
 lib/python${MODPY_VERSION}/site-packages/redis/exceptions.py
 lib/python${MODPY_VERSION}/site-packages/redis/exceptions.pyc
-lib/python${MODPY_VERSION}/site-packages/redis/types.py
-lib/python${MODPY_VERSION}/site-packages/redis/types.pyc

[Update] databases/py-redis

[wen heping]

Hi,

   Here is an update of databases/py-redis to 2.4.1.

   Tested OK on Loongson and it has no regress tests.

   Comments ? OK ?

wen
Index: py-redis/Makefile
===
RCS file: /cvs/ports/databases/py-redis/Makefile,v
retrieving revision 1.4
diff -u -r1.4 Makefile
--- py-redis/Makefile   20 Mar 2011 11:45:29 -  1.4
+++ py-redis/Makefile   2 Jun 2011 13:21:33 -
@@ -2,7 +2,7 @@
 
 COMMENT =  Python interface to Redis
 
-MODPY_EGG_VERSION =2.2.4
+MODPY_EGG_VERSION =2.4.1
 DISTNAME = redis-${MODPY_EGG_VERSION}
 PKGNAME =  py-redis-${MODPY_EGG_VERSION}
 CATEGORIES =   databases
Index: py-redis/distinfo
===
RCS file: /cvs/ports/databases/py-redis/distinfo,v
retrieving revision 1.4
diff -u -r1.4 distinfo
--- py-redis/distinfo   20 Mar 2011 11:45:29 -  1.4
+++ py-redis/distinfo   2 Jun 2011 13:21:33 -
@@ -1,5 +1,5 @@
-MD5 (py-redis/redis-2.2.4.tar.gz) = P8KMOje5Hfp8ZXMPbPeC4A==
-RMD160 (py-redis/redis-2.2.4.tar.gz) = KQyGP82Ty8q1j9lECLo82hGBC3c=
-SHA1 (py-redis/redis-2.2.4.tar.gz) = CQ8SN1nY+FuVGJlvOcE/O8SeJOg=
-SHA256 (py-redis/redis-2.2.4.tar.gz) = 
Kj4CHoYXMq10yoAWAnasZgXdaxlsQZi8QPwLmEemHMo=
-SIZE (py-redis/redis-2.2.4.tar.gz) = 20822
+MD5 (py-redis/redis-2.4.1.tar.gz) = TRQ14wkXB/bQeEixzTLVxA==
+RMD160 (py-redis/redis-2.4.1.tar.gz) = /HHSN0GNEBHOaWpX7u2gWgtGsLs=
+SHA1 (py-redis/redis-2.4.1.tar.gz) = +lVH1CMSEkfQIApqNyFoESktqC8=
+SHA256 (py-redis/redis-2.4.1.tar.gz) = 
VqsI18iRTwRSYG5NwjJjThPRW0orcTZb7d3MxVgv3V4=
+SIZE (py-redis/redis-2.4.1.tar.gz) = 21738
Index: py-redis/pkg/PLIST
===
RCS file: /cvs/ports/databases/py-redis/pkg/PLIST,v
retrieving revision 1.2
diff -u -r1.2 PLIST
--- py-redis/pkg/PLIST  20 Mar 2011 11:45:29 -  1.2
+++ py-redis/pkg/PLIST  2 Jun 2011 13:21:33 -
@@ -1,4 +1,4 @@
-@comment $OpenBSD: PLIST,v 1.2 2011/03/20 11:45:29 rpointel Exp $
+@comment $OpenBSD$
 lib/python${MODPY_VERSION}/site-packages/redis/
 
lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/
 
lib/python${MODPY_VERSION}/site-packages/redis-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/PKG-INFO
@@ -13,3 +13,5 @@
 lib/python${MODPY_VERSION}/site-packages/redis/connection.pyc
 lib/python${MODPY_VERSION}/site-packages/redis/exceptions.py
 lib/python${MODPY_VERSION}/site-packages/redis/exceptions.pyc
+lib/python${MODPY_VERSION}/site-packages/redis/types.py
+lib/python${MODPY_VERSION}/site-packages/redis/types.pyc

Re: [Update] databases/py-redis

[David Coppa]

On Thu, Jun 2, 2011 at 3:25 PM, wen heping wenhep...@gmail.com wrote:
 Hi,

   Here is an update of databases/py-redis to 2.4.1.

   Tested OK on Loongson and it has no regress tests.

   Comments ? OK ?

yep. ok for me.

ciao,
david