New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pythonPackages.django_3: init at 3 #96604
Conversation
there's a lot of packages with similar breakages:
|
Every major version will have only a single LTS release, which is the one we typically keep in Nixpkgs. https://www.djangoproject.com/download/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi,
Sorry for the delay for this review (I am currently in holiday, mostly AFK).
Overall, I am OK with the PR. Just the GIS support is not valid as is, the patch used to indicate where to find libgdal / libgeos is not valid anymore for django-3 branch.
I gave you a patch that can be used instead.
Also there seems to be merge conflicts in pkgs/top-level/python-packages.nix
. This PR should probably be rebased to master.
|
||
patches = stdenv.lib.optional withGdal | ||
(substituteAll { | ||
src = ./1.10-gis-libs.template.patch; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This patch is not valid for django-3.
You can re-use the one I prepared when I did a django-3.0 PR (#92848)
diff -Nur a/django/contrib/gis/gdal/libgdal.py b/django/contrib/gis/gdal/libgdal.py
--- a/django/contrib/gis/gdal/libgdal.py 2020-07-09 22:34:05.330568948 +0100
+++ b/django/contrib/gis/gdal/libgdal.py 2020-07-09 22:35:08.679095615 +0100
@@ -14,7 +14,7 @@
from django.conf import settings
lib_path = settings.GDAL_LIBRARY_PATH
except (AttributeError, ImportError, ImproperlyConfigured, OSError):
- lib_path = None
+ lib_path = "@gdal@/lib/libgdal@extension@"
if lib_path:
lib_names = None
diff -Nur a/django/contrib/gis/geos/libgeos.py b/django/contrib/gis/geos/libgeos.py
--- a/django/contrib/gis/geos/libgeos.py 2020-07-09 22:34:05.331568941 +0100
+++ b/django/contrib/gis/geos/libgeos.py 2020-07-09 22:36:24.863526276 +0100
@@ -24,7 +24,7 @@
from django.conf import settings
lib_path = settings.GEOS_LIBRARY_PATH
except (AttributeError, ImportError, ImproperlyConfigured, OSError):
- lib_path = None
+ lib_path = "@geos@/lib/libgeos_c@extension@"
# Setting the appropriate names for the GEOS-C library.
if lib_path:
please rebase on the latest master. I sorted |
@FRidh I am back from my break. Have you had time to rework this PR, or do you want me to take over it? |
@lsix feel free to take it over! |
I have updated the work done here in #98463 I close this PR in favor of the new one. |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)