[Django] #26753: Make GDAL a hard dependency
Django
------------------------------------------------+------------------------
Reporter: claudep | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: GIS | Version: master
Severity: Normal | Keywords:
Triage Stage: Accepted | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
------------------------------------------------+------------------------
Read https://groups.google.com/forum/#!topic/geodjango/gD0-1SMOBqU
This will allow us to simplify code in several places.
--
Ticket URL: <https://code.djangoproject.com/ticket/26753>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
Reporter: claudep | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: GIS | Version: master
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by timgraham):
To ease running the test suite by casual contributors, I think it's worth
continuing to allow running the Django test suite without GDAL installed.
In that case, we could skip all `gis_tests` instead of selectively
skipping tests in that package.
--
Ticket URL: <https://code.djangoproject.com/ticket/26753#comment:1>
Django
Reporter: claudep | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: GIS | Version: master
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by claudep):
> I think it's worth continuing to allow running the Django test suite
without GDAL installed
Absolutely. We might replace or complement the `HAS_GEOS`/`HAS_GDAL` flags
by some `HAS_GEOLIBS` flag.
--
Ticket URL: <https://code.djangoproject.com/ticket/26753#comment:2>
Django
Reporter: claudep | Owner: claudep
Type: Cleanup/optimization | Status: assigned
Component: GIS | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by claudep):
* owner: nobody => claudep
* status: new => assigned
--
Ticket URL: <https://code.djangoproject.com/ticket/26753#comment:3>
Django
Reporter: claudep | Owner: claudep
Type: Cleanup/optimization | Status: assigned
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by claudep):
* has_patch: 0 => 1
Comment:
[https://github.com/django/django/pull/6777 PR]
--
Ticket URL: <https://code.djangoproject.com/ticket/26753#comment:4>
Django
Reporter: claudep | Owner: claudep
Type: | Status: assigned
Cleanup/optimization |
Component: GIS | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by timgraham):
* stage: Accepted => Ready for checkin
--
Ticket URL: <https://code.djangoproject.com/ticket/26753#comment:5>
Django
Reporter: claudep | Owner: claudep
Cleanup/optimization |
Component: GIS | Version: master
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Ready for
| checkin
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Claude Paroz <claude@…>):
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"f7a363ee1d2039824d95f35e54219e09c5af67b0" f7a363e]:
{{{
#!CommitTicketReference repository=""
revision="f7a363ee1d2039824d95f35e54219e09c5af67b0"
Fixed #26753 -- Made GDAL a required dependency for contrib.gis
Thanks Tim Graham for the review.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/26753#comment:6>
Django
Reporter: Claude Paroz | Owner: Claude
Type: | Paroz
Cleanup/optimization | Status: closed
Component: GIS | Version: master
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Ready for
| checkin
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Tim Graham <timograham@…>):
In [changeset:"7c6efb3233ce01db96384483cef47769fba58ce2" 7c6efb32]:
{{{
#!CommitTicketReference repository=""
revision="7c6efb3233ce01db96384483cef47769fba58ce2"
Refs #26753 -- Removed obsolete workarounds in GIS tests for lack of GDAL.
Missed in f7a363ee1d2039824d95f35e54219e09c5af67b0.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/26753#comment:7>