[3.12] Merge Ubuntu test matrices in CI (GH-121813) #122116
Conversation
| if: needs.check_source.outputs.run_tests == 'true' | ||
| strategy: | ||
| matrix: | ||
| free-threading: |
There was a problem hiding this comment.
There's no free-threading in 3.12.
The Windows 3.12 backport included free-threading things in reusable-windows.yml but build.yml didn't mention it.
There was a problem hiding this comment.
... because the #121776 backport hasn't been merged yet :)
But that comments out the free-threading: true job. Do the same here?
There was a problem hiding this comment.
But that comments out the
free-threading: truejob. Do the same here?
Yes! I forgot about that one, only fixed the merge conflicts. I left it commented out to facilitate any future backports, reducing possible conflicts.
There was a problem hiding this comment.
I was wondering why that doesn't fail the CI and saw this:
configure: WARNING: unrecognized options: --disable-gil@hugovk any reason invalid flags don't crash the build process?
There was a problem hiding this comment.
Commented out that matrix factor. Marking this thread as resolved.
There was a problem hiding this comment.
I was wondering why that doesn't fail the CI and saw this:
configure: WARNING: unrecognized options: --disable-gil@hugovk any reason invalid flags don't crash the build process?
Let's ask @erlend-aasland
Co-authored-by: Hugo van Kemenade <[email protected]>. (cherry picked from commit a3f7db9) Co-authored-by: Sviatoslav Sydorenko <[email protected]>
webknjaz
force-pushed
the
backport-a3f7db9-3.12
branch
from
e1254a8 to
1d08251
Compare
|
Thanks! |
2 participants
.
(cherry picked from commit a3f7db9)