Re: [Django] #5437: Make GDAL optional for GIS test runner

[Django]

#5437: Make GDAL optional for GIS test runner
-+
 Reporter:  rcoup|Owner:  vlal
 Type:  Bug  |   Status:  closed
Component:  GIS  |  Version:  master
 Severity:  Normal   |   Resolution:  wontfix
 Keywords:  testing  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by timo):

 * status:  assigned => closed
 * resolution:   => wontfix


Comment:

 Given the lack of interest, I'm going to close this as "won't fix."

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.bc5e3e103fd35ebb2aee495a08e01134%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #5437: Make GDAL optional for GIS test runner

[Django]

#5437: Make GDAL optional for GIS test runner
-+
 Reporter:  rcoup|Owner:  vlal
 Type:  Bug  |   Status:  assigned
Component:  GIS  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  testing  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by vlal):

 Posted on django-dev mailing list [https://groups.google.com/forum/#!topic
 /django-developers/jr6WQ94JRoQ here]

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.0949ee0ef91baed4bfafb6c45583d07f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #5437: Make GDAL optional for GIS test runner

[Django]

#5437: Make GDAL optional for GIS test runner
-+
 Reporter:  rcoup|Owner:  vlal
 Type:  Bug  |   Status:  assigned
Component:  GIS  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  testing  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by vlal):

 * cc: vlal (added)


-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.f1c19517fddf43f8b2e09ff97b13f012%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #5437: Make GDAL optional for GIS test runner

[Django]

#5437: Make GDAL optional for GIS test runner
-+
 Reporter:  rcoup|Owner:  vlal
 Type:  Bug  |   Status:  assigned
Component:  GIS  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  testing  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by claudep):

 Thanks for the patch. But frankly, when I see that, I'm really tempted to
 make GDAL a required part of our GIS infrastructure. I think that in the
 past, GDAL was not so easily installable, but that limitation may be moot
 now. It might be worth debating this on django-dev mailing list.

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.abb9780dc611c91424c8b451fe302bf0%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #5437: Make GDAL optional for GIS test runner

[Django]

#5437: Make GDAL optional for GIS test runner
-+
 Reporter:  rcoup|Owner:  vlal
 Type:  Bug  |   Status:  assigned
Component:  GIS  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  testing  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by vlal):

 * owner:  nobody => vlal
 * status:  new => assigned


-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.0281a010ed90e875e3a0e537801138d9%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #5437: Make GDAL optional for GIS test runner

[Django]

#5437: Make GDAL optional for GIS test runner
-+
 Reporter:  rcoup|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  GIS  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  testing  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by vlal):

 Issued [https://github.com/django/django/pull/2516 Pull Request #5437].
 Verified tests either pass or skip when necessary.

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.77b84ad26712e864750747f8b3cda69e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #5437: Make GDAL optional for GIS test runner

[Django]

#5437: Make GDAL optional for GIS test runner
-+
 Reporter:  rcoup|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  GIS  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  testing  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by Alex):

 * stage:  Unreviewed => Accepted


-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.4bbe429cb2383596fe7f7a6a256943e4%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #5437: Make GDAL optional for GIS test runner

[Django]

#5437: Make GDAL optional for GIS test runner
-+--
 Reporter:  rcoup|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  GIS  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  testing  | Triage Stage:  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+--
Changes (by shai):

 * status:  closed => new
 * severity:   => Normal
 * type:   => Bug
 * version:  gis => master
 * easy:   => 0
 * keywords:  sprint => testing
 * ui_ux:   => 0
 * resolution:  fixed =>


Comment:

 This has apparently made a comeback; when running the tests without GDAL
 on master now, I get:
 {{{#!python
 Importing application geoadmin
 Traceback (most recent call last):
   File "runtests.py", line 379, in 
 options.failfast, args)
   File "runtests.py", line 198, in django_tests
 state = setup(verbosity, test_labels)
   File "runtests.py", line 172, in setup
 app_config.import_models(apps.all_models[app_config.label])
   File ".../django/apps/base.py", line 160, in import_models
 self.models_module = import_module(models_module_name)
   File "/usr/lib/python2.7/importlib/__init__.py", line 37, in
 import_module
 __import__(name)
   File ".../django/contrib/gis/tests/geoadmin/models.py", line 19, in
 
 admin.site.register(City, admin.OSMGeoAdmin)
 AttributeError: 'module' object has no attribute 'OSMGeoAdmin'
 }}}

 OSMGeoAdmin requires GDAL, and it is used unconditionally in a test.

 As far as I could see, this goes back to the introduction of the
 `geoadmin/models.py` file, back in September 2011, so although it is a
 regression, it has probably been this way since 1.4. On one hand, this
 means it is not a release blocker; on the other hand, this probably
 justifies backporting the fix to all the live versions.

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.e56f425f5cbac2ffce56c60fb62d52d4%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.