limpyd / redis-limpyd

Provide an easy way to store python objects in Redis, without losing the power and the control of the Redis API
https://redis-limpyd.readthedocs.org/
Do What The F*ck You Want To Public License
72 stars 11 forks source link

Remove the "set" cast in the setup.py #112

Closed clemfromspace closed 6 years ago

clemfromspace commented 6 years ago

The set cast on this line (https://github.com/limpyd/redis-limpyd/blob/master/setup.py#L23) in the setup.py file make this package uninstallable when setuptools version is >= 38.2.3

See also:

And also, hello @twidi :)

This is also the case in the related redis-limpyd-extensions package https://github.com/limpyd/redis-limpyd-extensions

twidi commented 6 years ago

In fact we should keep the set (to remove duplicates) and then convert it to a list.

But mainly, this setup.py should be should be rewritten from the ground up as explained (in french) here: https://twidi.github.io/python-packaging-talk/fr / http://pyvideo.org/pycon-fr-2017/les-aventuriers-du-packaging-perdu.html

But this will be for another time.

The important thing is that you just said to me that all of my packages using this way of managing dependencies are broken? I am sad, and it'w the worse time in two years for me, as I work 7/7 :(

clemfromspace commented 6 years ago

Yeah, we got this nice surprise yesterday on our main package since the mkvirtualenv install the latest version of setuptools by default (and there is no options to set the version). We fixed it on our main package, but there is some dependencies (limpyd and others maybe) who still use the set. So no other choices to downgrade setuptools after creating the virtualenv for the moment.

But thanks for the link, I will update our package accordingly :)

twidi commented 6 years ago

I think this will also break the next deployment of Isshub. Thanks for letting me know, better before deploying a fix in a rush than "oh f*** what's going on!!?!??!?!?!?" ;)

twidi commented 6 years ago

Fixed in #113 and upcoming version 1.0