Skip to content

Commit 5d92611

Browse files
authored
Use psycopg[binary] on all platforms (#262)
Since: - the binary wheel package is much faster to install than the compiled from source sdist, which is beneficial in an interactive tutorial scenario - the platform conditionals result in a more complicated requirements file (and confusing pip output about ignoring the `psycopg` package) which isn't ideal for beginners following the tutorial - it results in more deterministic app/dependencies (since the libpq version is now bundled with the package rather than being dynamically linked), and whilst this is a trade-off, it's more in keeping with the modern app approach of pinning dependencies using lockfiles etc GUS-W-19240129.
1 parent 3ca0fad commit 5d92611

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

requirements.txt

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,5 @@ gunicorn>=23,<24
33
dj-database-url>=3,<4
44
whitenoise>=6,<7
55

6-
# Uncomment these lines to use a Postgres database. Both are needed, since in production
7-
# (which uses Linux) we want to install from source, so that security updates from the
8-
# underlying Heroku stack image are picked up automatically, thanks to dynamic linking.
9-
# On other platforms/in development, the precompiled binary package is used instead, to
10-
# speed up installation and avoid errors from missing libraries/headers.
11-
#psycopg[c]; sys_platform == "linux"
12-
#psycopg[binary]; sys_platform != "linux"
6+
# Uncomment to use a Postgres database.
7+
#psycopg[binary]

0 commit comments

Comments
 (0)