Opened 11 years ago

Closed 10 years ago

#8768 closed task (Fixed)

Migrate Python bindings to oursql module

Reported by: Raymond Wagner Owned by: Raymond Wagner
Priority: major Milestone: 0.25
Component: Bindings - Python Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no


The current module used by the bindings for MySQL access is not supported by Python 2.7. 2.7 was only release in July, so it will take some time for distros to push this out to their stable packages, but this will likely become an issue at some point before the 0.25 release. If necessary, this may be backported to 0.24.

Oursql is not readily available in package repositories, so this will have an intermediate layer allowing fallback to the old module if on 2.6.

MythArchive? will be similarly affected, and should be altered to access the database through the bindings.

Change History (7)

comment:1 Changed 11 years ago by Raymond Wagner

Status: newaccepted

comment:2 Changed 11 years ago by stuartm

Milestone: 0.25

Milestone 0.25 deleted

comment:3 Changed 10 years ago by Raymond Wagner

Initial commit at r27184. Support added, but currently disabled pending . The 'ENABLE_OURSQL' environmental variable must be set to 'true' to enable it.

comment:4 Changed 10 years ago by Raymond Wagner

(In [27327]) Fix mirobridge to pass storagegroup directory name through unchanged if it is already unicode. Closes #9279 for now. Refs #8768. If this is going to be a recurring issue, it may be better to for all strings outputted by the MySQLdb module to unicode as well for consistency, and deal with whatever code needs to be fixed as a result.

comment:5 Changed 10 years ago by Raymond Wagner

Ticket locked: set

comment:6 Changed 10 years ago by Raymond Wagner

Ticket locked: unset

comment:7 Changed 10 years ago by Raymond Wagner

Resolution: Fixed
Status: acceptedclosed

Closing this one as fixed. The ENABLE_OURSQL flag is still required, and will be removed once the next version is released, incorporating the fix described above.

Note: See TracTickets for help on using tickets.