Skip to content

ci: Add test for minimum supported debug build of Python#5852

Open
Embers-of-the-Fire wants to merge 2 commits intoPyO3:mainfrom
Embers-of-the-Fire:ci-debug-min-version
Open

ci: Add test for minimum supported debug build of Python#5852
Embers-of-the-Fire wants to merge 2 commits intoPyO3:mainfrom
Embers-of-the-Fire:ci-debug-min-version

Conversation

@Embers-of-the-Fire
Copy link
Contributor

As per comment in #5847.

Makes sense. We should probably also add a Py_REF_DEBUG build to CI on the lowest supported Python version.

Note that the original debug build downloads prebuilt standalone binaries from python-build-standalone, but the prebuilt binaries are limited (for old versions). In the latest release there's only 3.10.19 available, and I could not find a 3.7 build in all releases.
This PR pins it to 3.10.19-3.14.3 from 20260211 release.

@ngoldbaum
Copy link
Contributor

In the latest release there's only 3.10.19 available

That's fine, 3.10 will be the minimum supported version soon enough. We're already way overdue for dropping 3.7 and 3.8 support.

@ngoldbaum
Copy link
Contributor

ngoldbaum commented Mar 3, 2026

I'll defer to @davidhewitt on whether it's OK to add another CI job to the full CI matrix. I added a ci-build-full label to make the new job actually run in this PR. You'll need to push another commit to trigger the tests again.

@ngoldbaum
Copy link
Contributor

The docs build error should go away once #5854 goes in.

@Embers-of-the-Fire
Copy link
Contributor Author

The error seems to be irrelavent with the CI :(

@davidhewitt
Copy link
Member

I think #5859 will fix the CI issues.

I am also looking at #5447 which should make this PR simpler to land & maintain.

@Embers-of-the-Fire
Copy link
Contributor Author

I am also looking at #5447 which should make this PR simpler to land & maintain.

:) agreed. Relying on curls is less than ideal. I'll have an eye on that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants