Skip to content

gh-136852: Emscripten: Add PYTHON_NODE_VERSION environment variable - #136853

Merged
ambv merged 1 commit into
python:mainfrom
hoodmane:emscripten-node-version
Jul 20, 2025
Merged

gh-136852: Emscripten: Add PYTHON_NODE_VERSION environment variable#136853
ambv merged 1 commit into
python:mainfrom
hoodmane:emscripten-node-version

Conversation

@hoodmane

@hoodmanehoodmane commented Jul 20, 2025

Copy link
Copy Markdown
Contributor

To choose the node version we use. Together with:
python/buildmaster-config#614closes#136852.

@ambv
ambv merged commit aec7f5f into python:mainJul 20, 2025
77 of 79 checks passed
@ambv

ambv commented Jul 20, 2025

Copy link
Copy Markdown
Contributor

Does this need backporting to 3.14?

@hoodmane

Copy link
Copy Markdown
ContributorAuthor

Yes I think it would be good to backport this.

@hoodmanehoodmane added the needs backport to 3.14 bugs and security fixes label Jul 21, 2025
@miss-islington-app

Copy link
Copy Markdown

Thanks @hoodmane for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

@hoodmane
hoodmane deleted the emscripten-node-version branch July 21, 2025 08:30
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 21, 2025
…able (pythonGH-136853)
To choose the node version we use. Together with:
python/buildmaster-config#614closespythonGH-136852.
(cherry picked from commit aec7f5f)
Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
@bedevere-app

Copy link
Copy Markdown

GH-136907 is a backport of this pull request to the 3.14 branch.

@bedevere-appbedevere-appBot removed the needs backport to 3.14 bugs and security fixes label Jul 21, 2025
@hoodmane

Copy link
Copy Markdown
ContributorAuthor

Hmm well again it doesn't seem to work quite.

hoodmane added a commit to hoodmane/cpython that referenced this pull request Jul 21, 2025
We need to get rid of some whitespace to make this work.
ambv pushed a commit that referenced this pull request Jul 21, 2025
We need to get rid of some whitespace to make this work.
hoodmane added a commit to miss-islington/cpython that referenced this pull request Jul 21, 2025
We need to get rid of some whitespace to make this work.
ambv pushed a commit that referenced this pull request Jul 21, 2025
…iable (GH-136853) (GH-136907)
To choose the node version we use. Together with:
python/buildmaster-config#614closesGH-136852.
(cherry picked from commit aec7f5f)
Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
We need to get rid of some whitespace to make this work.
Agent-Hellboy pushed a commit to Agent-Hellboy/cpython that referenced this pull request Aug 19, 2025
Agent-Hellboy pushed a commit to Agent-Hellboy/cpython that referenced this pull request Aug 19, 2025
We need to get rid of some whitespace to make this work.
kumaraditya303 pushed a commit to miss-islington/cpython that referenced this pull request Sep 9, 2025
…nt variable (pythonGH-136853) (pythonGH-136907)
To choose the node version we use. Together with:
python/buildmaster-config#614closespythonGH-136852.
(cherry picked from commit aec7f5f)
Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
Sign up for freeto 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.

Emscripten buildbot should run against node 24

3 participants

@hoodmane@ambv@AA-Turner