Skip to content

Add support for python 3.13 and 3.13t - #3

Merged
yabirgb merged 6 commits into
rotki:mainfrom
yabirgb:python313
Jul 7, 2026
Merged

Add support for python 3.13 and 3.13t#3
yabirgb merged 6 commits into
rotki:mainfrom
yabirgb:python313

Conversation

@yabirgb

@yabirgb yabirgb commented Jul 6, 2026

Copy link
Copy Markdown
Member
  • syncs the code with python 3.13
  • builds both 3.13 and 3.13t branches

@yabirgb
yabirgb force-pushed the python313 branch 2 times, most recently from bc94688 to 45dbc2f Compare July 7, 2026 06:45
yabirgb added 2 commits July 7, 2026 09:02
Defining Py_BUILD_CORE globally makes Windows treat Python API data as core symbols instead of imported DLL symbols. The cp313t Windows wheel then fails to link references such as _Py_NoneStruct and PyExc_TypeError from objects that only use the public API.\n\nThe _sqlite sources that need CPython internals already define Py_BUILD_CORE_MODULE before including Python headers, which still enables internal headers while preserving dynamic module semantics on Windows.
Comment thread .github/workflows/build.yml Outdated
Comment on lines +171 to +176
$env:CIBW_BUILD = "cp313-*"
$env:LINK = "python313.lib"
python -m cibuildwheel --output-dir dist

$env:CIBW_BUILD = "cp313t-*"
$env:LINK = "python313t.lib"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any specific reason why we set the environment here
like this and we don't split it into two steps with their own env that each call their own cibuildwheel?

Also do they have to be separate can we have a single command build both?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reason it was there was because of a flag that I later removed and required to build against a specific DLL. I managed to remove the flag and now that specific dll is not needed for windows since the build can autodetect what is needed

Comment thread uv.lock Outdated
Comment thread pyproject.toml Outdated
@yabirgb
yabirgb merged commit 1f13d5a into rotki:main Jul 7, 2026
10 checks passed
@yabirgb
yabirgb deleted the python313 branch July 7, 2026 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants