Skip to content

Commit

Permalink
Upgrade dependencies and test matrix for Python 3.13.
Browse files Browse the repository at this point in the history
Add new migration for updated le-utils.
Add cgi forward port compatibility patch.
  • Loading branch information
rtibbles committed Oct 24, 2024
1 parent 068f25b commit 406aca5
Show file tree
Hide file tree
Showing 7 changed files with 110 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/tox.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
strategy:
max-parallel: 5
matrix:
python-version: [3.6, 3.7, 3.8, 3.9, '3.10', '3.11', '3.12']
python-version: [3.6, 3.7, 3.8, 3.9, '3.10', '3.11', '3.12', '3.13']

steps:
- uses: actions/checkout@v4
Expand Down
42 changes: 42 additions & 0 deletions kolibri/core/content/migrations/0038_alter_localfile_extension.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
# Generated by Django 3.2.25 on 2024-10-24 22:56
from django.db import migrations
from django.db import models


class Migration(migrations.Migration):

dependencies = [
("content", "0037_add_bloompub_preset"),
]

operations = [
migrations.AlterField(
model_name="localfile",
name="extension",
field=models.CharField(
blank=True,
choices=[
("mp4", "MP4 Video"),
("webm", "WEBM Video"),
("vtt", "VTT Subtitle"),
("mp3", "MP3 Audio"),
("pdf", "PDF Document"),
("jpg", "JPG Image"),
("jpeg", "JPEG Image"),
("png", "PNG Image"),
("gif", "GIF Image"),
("json", "JSON"),
("svg", "SVG Image"),
("perseus", "Perseus Exercise"),
("graphie", "Graphie Exercise"),
("zip", "HTML5 Zip"),
("h5p", "H5P"),
("zim", "ZIM"),
("epub", "ePub Document"),
("bloompub", "Bloom Document"),
("bloomd", "Bloom Document"),
],
max_length=40,
),
),
]
47 changes: 47 additions & 0 deletions kolibri/utils/compat_cgi.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
"""
A minimal port of the removed cgi module for use in Python 3.13.
Only imports the specific parts of the module that are used by Django.
Informed by the PR that removed its use in Django:
https://github.com/django/django/pull/15679
"""
from django.utils.regex_helper import _lazy_re_compile


def _parseparam(s):
while s[:1] == ";":
s = s[1:]
end = s.find(";")
while end > 0 and (s.count('"', 0, end) - s.count('\\"', 0, end)) % 2:
end = s.find(";", end + 1)
if end < 0:
end = len(s)
f = s[:end]
yield f.strip()
s = s[end:]


def parse_header(line):
"""
Parse a Content-type like header.
Return the main content-type and a dictionary of options.
"""
parts = _parseparam(";" + line)
key = parts.__next__()
pdict = {}
for p in parts:
i = p.find("=")
if i >= 0:
name = p[:i].strip().lower()
value = p[i + 1 :].strip()
if len(value) >= 2 and value[0] == value[-1] == '"':
value = value[1:-1]
value = value.replace("\\\\", "\\").replace('\\"', '"')
pdict[name] = value
return key, pdict


boundary_re = _lazy_re_compile(rb"[ -~]{0,200}[!-~]")


def valid_boundary(boundary):
return boundary_re.fullmatch(boundary) is not None
14 changes: 14 additions & 0 deletions kolibri/utils/env.py
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,19 @@ def monkey_patch_distutils():
sys.modules["distutils.version"] = module


def forward_port_cgi_module():
"""
Forward ports the required parts of the removed cgi module.
This can be removed when we upgrade to a version of Django that is Python 3.13 compatible.
"""
if sys.version_info < (3, 13):
return
from importlib import import_module

module = import_module("kolibri.utils.compat_cgi")
sys.modules["cgi"] = module


def set_env():
"""
Sets the Kolibri environment for the CLI or other application worker
Expand All @@ -120,6 +133,7 @@ def set_env():
"""
monkey_patch_markdown()
monkey_patch_distutils()
forward_port_cgi_module()

from kolibri import dist as kolibri_dist # noqa

Expand Down
4 changes: 2 additions & 2 deletions requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@ django-mptt==0.14.0
requests==2.27.1
cheroot==10.0.1
magicbus==4.1.2
le-utils==0.2.6
le-utils==0.2.8
jsonfield==3.1.0
morango==0.8.1
morango==0.8.2
tzlocal==4.2
pytz==2024.1
python-dateutil==2.9.0.post0
Expand Down
3 changes: 2 additions & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -100,8 +100,9 @@ def run(self):
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
"Programming Language :: Python :: 3.12",
"Programming Language :: Python :: 3.13",
"Programming Language :: Python :: Implementation :: PyPy",
],
cmdclass={"install_scripts": gen_windows_batch_files},
python_requires=">=3.6, <3.13",
python_requires=">=3.6, <3.14",
)
3 changes: 2 additions & 1 deletion tox.ini
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
[tox]
envlist = py{3.6,3.7,3.8,3.9,3.10,3.11,3.12}, postgres
envlist = py{3.6,3.7,3.8,3.9,3.10,3.11,3.12,3.13}, postgres

[testenv]
usedevelop = True
Expand All @@ -21,6 +21,7 @@ basepython =
py3.10: python3.10
py3.11: python3.11
py3.12: python3.12
py3.13: python3.13
deps =
-r{toxinidir}/requirements/test.txt
-r{toxinidir}/requirements/base.txt
Expand Down

0 comments on commit 406aca5

Please sign in to comment.