Skip to content

build: fix build with Python 3.12 - #50582

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
lpinca:fix/build-with-python-3-12
Nov 8, 2023
Merged

build: fix build with Python 3.12#50582
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
lpinca:fix/build-with-python-3-12

Conversation

@lpinca

Copy link
Copy Markdown
Member

Replace distutils.version.StrictVersion with
packaging.version.Version.

@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Nov 6, 2023
@lpinca
lpincaforce-pushed the fix/build-with-python-3-12 branch from 66e9326 to 676e9b4CompareNovember 6, 2023 20:09
@lpincalpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 6, 2023
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 6, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Replace `distutils.version.StrictVersion` with
`packaging.version.Version`.
Refs: nodejs#50209 (comment)
@lpinca
lpincaforce-pushed the fix/build-with-python-3-12 branch from 676e9b4 to 7ce12adCompareNovember 7, 2023 07:33
@lpincalpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 7, 2023
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 7, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@richardlau

Copy link
Copy Markdown
Member

Like #50209 this depends on #50380.
cc @nodejs/backporters

@legendecas

Copy link
Copy Markdown
Member

Can we fast-track this? configure on the main branch with python3.12 installed on the system will fail.

@lpincalpinca added the fast-track PRs that do not need to wait for 48 hours to land. label Nov 7, 2023
@github-actions

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @lpinca. Please 👍 to approve.

@lpincalpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 8, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 8, 2023
@nodejs-github-bot
nodejs-github-bot merged commit 95534ad into nodejs:mainNov 8, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 95534ad

@lpinca
lpinca deleted the fix/build-with-python-3-12 branch November 8, 2023 21:00
targos pushed a commit that referenced this pull request Nov 11, 2023
Replace `distutils.version.StrictVersion` with
`packaging.version.Version`.
Refs: #50209 (comment)
PR-URL: #50582
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
@targostargos mentioned this pull request Nov 12, 2023
targos pushed a commit that referenced this pull request Nov 14, 2023
Replace `distutils.version.StrictVersion` with
`packaging.version.Version`.
Refs: #50209 (comment)
PR-URL: #50582
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
UlisesGascon pushed a commit that referenced this pull request Dec 11, 2023
Replace `distutils.version.StrictVersion` with
`packaging.version.Version`.
Refs: #50209 (comment)
PR-URL: #50582
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
@UlisesGasconUlisesGascon mentioned this pull request Dec 12, 2023
joshuafried pushed a commit to joshuafried/node that referenced this pull request Sep 13, 2024
Replace `distutils.version.StrictVersion` with
`packaging.version.Version`.
Refs: nodejs/node#50209 (comment)
PR-URL: nodejs/node#50582
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
aduh95 pushed a commit that referenced this pull request Nov 2, 2024
Replace `distutils.version.StrictVersion` with
`packaging.version.Version`.
Refs: #50209 (comment)
PR-URL: #50582
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildIssues and PRs related to build files or the CI.fast-trackPRs that do not need to wait for 48 hours to land.needs-ciPRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lpinca@nodejs-github-bot@richardlau@legendecas@aduh95