[PATCH v2] Move django-debug-toolbar to requirements-dev.txt

Daniel Axtens dja at axtens.net
Thu Aug 25 19:47:16 AEST 2016


Don't install django-debug-toolbar as part of test requirements,
but as part of dev requirements. This means we can install a more
recent version and not have to worry about conflicts with old
versions of Django interfering.

Signed-off-by: Daniel Axtens <dja at axtens.net>
---

This is my suggestion at a better way to approach the
sqlparse/django-debug-toolbar mess.

v2: tag the import with noqa so flake8 doesn't complain.

---
 patchwork/settings/dev.py | 34 +++++++++++++++++++---------------
 requirements-dev.txt      |  1 +
 requirements-test.txt     |  2 +-
 3 files changed, 21 insertions(+), 16 deletions(-)

diff --git a/patchwork/settings/dev.py b/patchwork/settings/dev.py
index b78a0bfeb0c8..bcc1db7200eb 100644
--- a/patchwork/settings/dev.py
+++ b/patchwork/settings/dev.py
@@ -66,21 +66,25 @@ PASSWORD_HASHERS = ['django.contrib.auth.hashers.MD5PasswordHasher']
 # django-debug-toolbar
 
 if django.VERSION >= (1, 7):
-    INSTALLED_APPS += [
-        'debug_toolbar'
-    ]
-
-    DEBUG_TOOLBAR_PATCH_SETTINGS = False
-
-    # This should go first in the middleware classes
-    MIDDLEWARE_CLASSES = [
-        'debug_toolbar.middleware.DebugToolbarMiddleware',
-    ] + MIDDLEWARE_CLASSES
-
-    INTERNAL_IPS = [
-        '127.0.0.1', '::1',
-        '172.17.0.1'
-    ]
+    try:
+        import debug_toolbar  # noqa
+        INSTALLED_APPS += [
+            'debug_toolbar'
+        ]
+
+        DEBUG_TOOLBAR_PATCH_SETTINGS = False
+
+        # This should go first in the middleware classes
+        MIDDLEWARE_CLASSES = [
+            'debug_toolbar.middleware.DebugToolbarMiddleware',
+        ] + MIDDLEWARE_CLASSES
+
+        INTERNAL_IPS = [
+            '127.0.0.1', '::1',
+            '172.17.0.1'
+        ]
+    except:
+        pass
 
 
 #
diff --git a/requirements-dev.txt b/requirements-dev.txt
index d44ebc697bed..947c8d32744e 100644
--- a/requirements-dev.txt
+++ b/requirements-dev.txt
@@ -1,2 +1,3 @@
 Django>=1.8,<1.10
+django-debug-toolbar==1.5
 -r requirements-test.txt
diff --git a/requirements-test.txt b/requirements-test.txt
index cfc242f647f8..89b466a4aca0 100644
--- a/requirements-test.txt
+++ b/requirements-test.txt
@@ -1,6 +1,6 @@
 mysqlclient==1.3.7  # replace this with psycopg2 for a PostgreSQL backend
-django-debug-toolbar==1.4
 python-dateutil>2.0,<3.0
 selenium>2.0,<3.0
 djangorestframework>=3.3,<3.4
 drf-nested-routers>=0.11.1,<0.12
+sqlparse
-- 
2.7.4



More information about the Patchwork mailing list