Merge pull request #408 from newearthmartin/deps

moving optional dependencies

Author: newearthmartin

README.rst

@@ -58,9 +58,20 @@ Currently, django-newsletter officially supports Django 4.2.x LTS, 5.0.x and 5.1
 
 Requirements
 ============
-Please refer to `requirements.txt <http://github.com/jazzband/django-newsletter/blob/master/requirements.txt>`_
+Please refer to `setup.py <http://github.com/jazzband/django-newsletter/blob/master/setup.py>`_
 for an updated list of required packages.
 
+Also, you will need to install either one of these to use as a thumbnail engine:
+
+* sorl-thumbnail
+* easy-thumbnails
+
+Additional dependencies that need to be installed separately:
+
+* python-ldap: for importing ldif files
+* unicodecsv: for importing csv files
+* python-card-me: for importing vCard files
+
 Tests
 ==========
 Fairly extensive tests are available for internal frameworks, web

setup.py

@@ -44,12 +44,18 @@
     long_description=README,
     install_requires=[
         "Django>=4.2",
-        "python-card-me<1.0",
-        "python-ldap",
         "chardet",
-        "unicodecsv<0.15",
         "Pillow",
     ],
+    extras_require={
+        "test": [
+            "python-ldap",
+            "unicodecsv",
+            "python-card-me",
+            "django-tinymce",
+            "django-imperavi"
+        ],
+    },
     author='Mathijs de Bruin',
     author_email='[email protected]',
     url='http://github.com/jazzband/django-newsletter/',

tox.ini

@@ -7,15 +7,15 @@ envlist =
 [testenv]
 deps =
     coverage
-    django-imperavi
-    django-tinymce
     pytz
     webtest
     django-webtest
     dj42: Django>=4.2,<4.3
     dj51: Django>=5.1,<5.2
     dj52: Django>=5.2,<5.3
     djmain: https://github.com/django/django/archive/main.tar.gz
+extras =
+    test
 usedevelop = True
 ignore_outcome =
     djmain: True