Edgewall Software

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#11969 closed task (fixed)

Specify minimum supported version of PySqlite — at Version 5

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Milestone: 1.1.4
Component: database backend Version:
Severity: normal Keywords: sqlite pysqlite
Cc: Branch:
Release Notes:

Minimum version of pysqlite is enforced to be 2.4.1.

API Changes:
Internal Changes:

Description (last modified by Ryan J Ollos)

Python 2.5 include sqlite3. The proposed change is that we adopt as the minimum supported version of PySqlite the version of PySqlite provided in the sqlite3 module by the minimum supported version of Python. For Python 2.6 this will be PySqlite 2.4.1.

Change History (5)

comment:1 by Ryan J Ollos, 9 years ago

I was confused by the code, SQLite 2.x is already not supported. However, we can clarify supported PySqlite versions.

Last edited 9 years ago by Ryan J Ollos (previous) (diff)

comment:2 by Ryan J Ollos, 9 years ago

Description: modified (diff)
Milestone: undecided1.1.5
Owner: set to Ryan J Ollos
Status: newassigned
Summary: Remove support for SQLite < 3.0Specify minimum supported version of PySqlite

comment:3 by Ryan J Ollos, 9 years ago

With sqlite3 included in the standard library in Python 2.5 and later I can't foresee a scenario in which the PySqlite bindings aren't available in a valid Python installation. That motivated some cleanup of the code and elimination of the have_sqlite check. Proposed changes in log:rjollos.git:t11969.

Last edited 9 years ago by Ryan J Ollos (previous) (diff)

comment:4 by Jun Omae, 9 years ago

Looks good to me. Confirmed on both Python 2.6 with bundled pysqlite 2.4.1 (sqlite3 is used) and Python 2.6 with new installed pysqlite 2.6.3 (pysqlite2.dbapi2 is used).

comment:5 by Ryan J Ollos, 9 years ago

Milestone: 1.1.51.1.4
Release Notes: modified (diff)
Resolution: fixed
Status: assignedclosed

Thanks for the review. Committed in [13805].

Note: See TracTickets for help on using tickets.