Opened 9 years ago

Closed 8 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

Description

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 9 years ago by Raymond Wagner

Status: newaccepted

comment:2 Changed 9 years ago by stuartm

Milestone: 0.25

Milestone 0.25 deleted

comment:3 Changed 9 years ago by Raymond Wagner

Initial commit at r27184. Support added, but currently disabled pending https://bugs.launchpad.net/oursql/+bug/672059 . The 'ENABLE_OURSQL' environmental variable must be set to 'true' to enable it.

comment:4 Changed 9 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 9 years ago by Raymond Wagner

Ticket locked: set

comment:6 Changed 9 years ago by Raymond Wagner

Ticket locked: unset

comment:7 Changed 8 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.