#33886: GeoIP module unnecessarily requires GDAL
-----------------------------------------+-------------------------------
Reporter: Aarni Koskela | Owner: Aarni Koskela
Type: Bug | Status: assigned
Component: GIS | Version: 4.1
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-----------------------------------------+-------------------------------
The commit
https://github.com/django/django/commit/31bef51d8ea5e3bf650de1495ae1f0566670b8f3
(in versions 4.1a1 forward) moved the import of
`django.contrib.gis.geos.Point` from the `def geos()` function to top-
level, with the unfortunate side-effect that importing the `GeoIP` module
now requires working GDAL libraries.
The GeoIP module should be usable without GDAL; naturally, returning a
GDAL geometry would require GDAL.
My suggested fix would be to revert the above commit; a late import should
be fine here.
--
Ticket URL: <https://code.djangoproject.com/ticket/33886>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/0107018263ed013a-0b7156c7-b4b9-4d0d-8e53-e8e6a3631409-000000%40eu-central-1.amazonses.com.
