search

mahmoudimus / sqlalchemy-citext

CITEXT type for SQLAlchemy
Other
29 stars 14 forks source link

Don't depend on psycopg2 #3

Closed dstufft closed 11 years ago

dstufft commented 11 years ago

Depending on psycopg2 makes this module unusable on PyPy. If the dependency was left off PyPy users could use psycopg2cffi (which implements a psycopg2 compatible api ontop of cffi which works with PyPy).

I'd really hate to need to fork this library just to delete a dependency :/

mahmoudimus commented 11 years ago

Makes sense. We should be assuming that you're using some kind of postgres adapter.

mahmoudimus commented 11 years ago

I'll fix soon - thanks for reporting @dstufft

dstufft commented 11 years ago

Awesome :sparkles: Saves me from needing to maintain my own branch.

dstufft commented 11 years ago

Awesome! Any idea when you'll make a new release?

mahmoudimus commented 11 years ago

Published to PyPI under 1.1.0: https://pypi.python.org/pypi/sqlalchemy-citext/1.1-0

mahmoudimus commented 11 years ago

Thanks for reporting :)

fake-name commented 3 years ago

This is an old issue, but https://github.com/mahmoudimus/sqlalchemy-citext/pull/19 broke pypy compat.

fake-name commented 2 years ago

Hello? Still hitting this.