Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#11609 closed (fixed)

Allow for long pointer format in django.contrib.gis.gdal.base

Reported by: rmkemker Owned by: nobody
Component: GIS Version: 1.1-beta
Severity: Keywords: gdal.base
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: yes Patch needs improvement: yes
Easy pickings: UI/UX:

Description

My apache webserver is outputting long pointer types for the ESRI Driver. Adding a check for the long format and then converting it to int allows for the webserver to work correctly.

Attachments (2)

base.py (1.2 KB) - added by rmkemker 6 years ago.
django.contrib.gis.gdal.base
base.2.py (1.1 KB) - added by starsareblueandfaraway 6 years ago.
You do not need to type-cast the long into an integer

Download all attachments as: .zip

Change History (10)

Changed 6 years ago by rmkemker

django.contrib.gis.gdal.base

comment:1 Changed 6 years ago by rmkemker

  • milestone set to 1.2
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 6 years ago by rmkemker

  • Version changed from 1.0 to 1.1-beta-1

comment:3 in reply to: ↑ description Changed 6 years ago by starsareblueandfaraway

Replying to rmkemker:

My apache webserver is outputting long pointer types for the ESRI Driver. Adding a check for the long format and then converting it to int allows for the webserver to work correctly.

I am having the same problem. Can someone please address this issue?

Changed 6 years ago by starsareblueandfaraway

You do not need to type-cast the long into an integer

comment:4 Changed 6 years ago by tobias

  • Has patch unset
  • Needs tests set
  • Patch needs improvement set

You need to provide a patch, not a copy of a source code file. http://docs.djangoproject.com/en/dev/internals/contributing/#patch-style

You will probably also need a test to demonstrate that there's a problem (and that your proposed fix actually fixes it).

comment:5 Changed 6 years ago by jcd

  • Component changed from Contrib apps to GIS
  • Triage Stage changed from Unreviewed to Accepted

comment:6 Changed 6 years ago by jbronn

  • Resolution set to fixed
  • Status changed from new to closed

(In [12006]) Fixed #11609 -- The check_pointer error checking routine and GDALBase._set_ptr are now able to handle the long pointer addresses used by some x86_64 platforms. Thanks, rmkemker, for the bug report.

comment:7 Changed 6 years ago by jbronn

(In [12007]) [1.1.X] Fixed #11609 -- The check_pointer error checking routine and GDALBase._set_ptr are now able to handle the long pointer addresses used by some x86_64 platforms. Thanks, rmkemker, for the bug report.

Backport of r12006 from trunk.

comment:8 Changed 4 years ago by jacob

  • milestone 1.2 deleted

Milestone 1.2 deleted

Note: See TracTickets for help on using tickets.
Back to Top