Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gh-93370 : Deprecate sqlite3.version and sqlite3.version_info #93482

Merged
merged 29 commits into from
Jun 7, 2022
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
5debfa5
remove version and version_info from sqlite3
rawwar Jun 3, 2022
c648144
Merge branch 'python:main' into fix-issue-93370
rawwar Jun 3, 2022
ee0cec7
📜🤖 Added by blurb_it.
blurb-it[bot] Jun 3, 2022
69e7410
used __getattr__ to raise warnings
rawwar Jun 3, 2022
6c2a8c8
removed repeated declaration
rawwar Jun 3, 2022
459a94c
including feedback
rawwar Jun 4, 2022
4a55ef3
move declaration to the start of the module
rawwar Jun 4, 2022
1a75e5f
fixed formatting issues
rawwar Jun 4, 2022
8584a03
extra lines removed
rawwar Jun 4, 2022
fa80dd6
Update Lib/sqlite3/__init__.py
rawwar Jun 4, 2022
f47ddf3
Update Lib/sqlite3/dbapi2.py
rawwar Jun 4, 2022
f033aea
Update Lib/sqlite3/dbapi2.py
rawwar Jun 4, 2022
49ea9e1
updated docs
rawwar Jun 4, 2022
451e9ef
Update Lib/sqlite3/__init__.py
rawwar Jun 4, 2022
591f2b9
Update Lib/sqlite3/dbapi2.py
rawwar Jun 4, 2022
3f4e967
Apply suggestions from code review
rawwar Jun 4, 2022
d70b247
update blurb with correct ref
rawwar Jun 4, 2022
ed50588
included my name in ACKS
rawwar Jun 4, 2022
8f06e31
Update Doc/library/sqlite3.rst
rawwar Jun 4, 2022
836cddf
adding tests for version and version_info deprec
rawwar Jun 4, 2022
2102bf0
added dbapi2 tests and updated docs
rawwar Jun 4, 2022
04dad03
use mod sqlite3 in sqlite3 doc
rawwar Jun 4, 2022
81cf7fa
Update Doc/library/sqlite3.rst
erlend-aasland Jun 5, 2022
964cf86
Update Doc/library/sqlite3.rst
erlend-aasland Jun 5, 2022
f00b352
added assertIn and assertEqual in tests
rawwar Jun 7, 2022
1920746
using assertEqual
rawwar Jun 7, 2022
46922f6
removed white space
rawwar Jun 7, 2022
45478cc
Apply suggestions from code review
rawwar Jun 7, 2022
bc7b463
Update Lib/test/test_sqlite3/test_dbapi.py
erlend-aasland Jun 7, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions Lib/sqlite3/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,3 +55,11 @@
"""

from sqlite3.dbapi2 import *
from sqlite3.dbapi2 import _deprecated_version_info, _deprecated_version
from warnings import warn
rawwar marked this conversation as resolved.
Show resolved Hide resolved

def __getattr__(name):
if name in deprecated_names:
warn(f"{name} is deprecated. Will be removed in python 3.14", DeprecationWarning, stacklevel=2)
return globals()[f"_deprecated_{name}"]
raise AttributeError(f"module {__name__} has no attribute {name}")
rawwar marked this conversation as resolved.
Show resolved Hide resolved
12 changes: 11 additions & 1 deletion Lib/sqlite3/dbapi2.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@
import collections.abc

from _sqlite3 import *
from _sqlite3 import _deprecated_version
from warnings import warn
rawwar marked this conversation as resolved.
Show resolved Hide resolved

paramstyle = "qmark"

Expand All @@ -45,7 +47,7 @@ def TimeFromTicks(ticks):
def TimestampFromTicks(ticks):
return Timestamp(*time.localtime(ticks)[:6])

version_info = tuple([int(x) for x in version.split(".")])
_deprecated_version_info = tuple([int(x) for x in _deprecated_version.split(".")])
sqlite_version_info = tuple([int(x) for x in sqlite_version.split(".")])

Binary = memoryview
Expand Down Expand Up @@ -85,3 +87,11 @@ def convert_timestamp(val):
# Clean up namespace

del(register_adapters_and_converters)

deprecated_names = ["version", "version_info"]
rawwar marked this conversation as resolved.
Show resolved Hide resolved

def __getattr__(name):
if name in deprecated_names:
warn(f"{name} is deprecated. Will be removed in python 3.14", DeprecationWarning, stacklevel=2)
return globals()[f"_deprecated_{name}"]
raise AttributeError(f"module {__name__} has no attribute {name}")
rawwar marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
deprecate sqlite3.version and sqlite3.version_info.
rawwar marked this conversation as resolved.
Show resolved Hide resolved
2 changes: 1 addition & 1 deletion Modules/_sqlite/module.c
Original file line number Diff line number Diff line change
Expand Up @@ -707,7 +707,7 @@ module_exec(PyObject *module)
goto error;
}

if (PyModule_AddStringConstant(module, "version", PYSQLITE_VERSION) < 0) {
if (PyModule_AddStringConstant(module, "_deprecated_version", PYSQLITE_VERSION) < 0) {
goto error;
}

Expand Down