pythonPackages.django-sites: fix build, flesh out comments

1 files changed, 17 insertions, 10 deletions

diff --git a/pkgs/development/python-modules/django-sites/default.nix b/pkgs/development/python-modules/django-sites/default.nix
index 63c61131261d3..2882e1dc2b93a 100644
--- a/pkgs/development/python-modules/django-sites/default.nix
+++ b/pkgs/development/python-modules/django-sites/default.nix
@@ -4,22 +4,29 @@ buildPythonPackage rec {
   pname = "django-sites";
   version = "0.10";
 
-  meta = {
-    description = ''
-      Alternative implementation of django "sites" framework
-      based on settings instead of models.
-    '';
-    homepage = https://github.com/niwinz/django-sites;
-    license = lib.licenses.bsd3;
-  };
-
   src = fetchPypi {
     inherit pname version;
     sha256 = "f6f9ae55a05288a95567f5844222052b6b997819e174f4bde4e7c23763be6fc3";
   };
+  # LICENSE file appears to be missing from pypi package, but expected by the installer
+  # https://github.com/niwinz/django-sites/issues/11
+  postPatch = ''
+    touch LICENSE
+  '';
 
   propagatedBuildInputs = [ django ];
 
-  # django.core.exceptions.ImproperlyConfigured: Requested settings, but settings are not configured. You must either define the environment variable DJANGO_SETTINGS_MODULE or call settings.configure() before accessing settings.
+  # required files for test don't seem to be included in pypi package, full source for 0.10
+  # version doesn't appear to be present on github
+  # https://github.com/niwinz/django-sites/issues/9
   doCheck = false;
+
+  meta = {
+    description = ''
+      Alternative implementation of django "sites" framework
+      based on settings instead of models.
+    '';
+    homepage = https://github.com/niwinz/django-sites;
+    license = lib.licenses.bsd3;
+  };
 }