Opened 12 years ago
Closed 8 years ago
#10760 closed Patch - Bug Fix (Won't Fix)
[PATCH] Python bindings: Relax mysql Python binding version
Reported by: | Owned by: | Lawrence Rust | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | Bindings - Python | Version: | Master Head |
Severity: | medium | Keywords: | mysql 1.2.3 |
Cc: | Ticket locked: | no |
Description
Commit 02162459d68a added a check for version 1.2.3 or later of the mysql Python binding. Unfortunately, this version only ships with Ubuntu 11.10 and later preventing all but the very latest Ubuntu systems from using any MythTV Python app.
Ubuntu 10.04 LTS ships with version 1.2.2 of the mysql binding and from basic testing this appears to work OK.
This patch relaxes the mysql Python binding version to 1.2,2 or later.
Attachments (1)
Change History (4)
Changed 12 years ago by
Attachment: | 0001-Python-bindings-Relax-mysql-Python-binding-version.patch added |
---|
comment:1 Changed 12 years ago by
comment:2 Changed 8 years ago by
Owner: | changed from Raymond Wagner to Lawrence Rust |
---|---|
Status: | new → assigned |
Lawrence, assigning your open patches to yourself. Feel free to unassign if you prefer a review.
comment:3 Changed 8 years ago by
Resolution: | → Won't Fix |
---|---|
Status: | assigned → closed |
The oldest Ubuntu LTS version is now 12.04, which ships with python-mysqldb version 1.2.3, so this patch is "past its sell-by date".
I know at the time I put that in there, I had good reason for it. There were problems trying to use MySQLdb 1.2.2 with Python 2.7, which was also a big part of the why I added oursql support, and something that wasn't fixed until 1.2.3. Searching around, I'm coming up blank with what those problems were, and a brief test with it now, it appears to work. I'll need to dig through IRC logs to try to remember why I did that before committing this.