Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 20, 2025

Conversation

hoodmane
Copy link
Contributor

@hoodmane hoodmane commented Jul 20, 2025

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

@hoodmane hoodmane requested a review from freakboy3742 as a code owner July 20, 2025 10:01
@ambv ambv merged commit aec7f5f into python:main Jul 20, 2025
77 of 79 checks passed
@ambv
Copy link
Contributor

ambv commented Jul 20, 2025

Does this need backporting to 3.14?

@hoodmane
Copy link
Contributor Author

Yes I think it would be good to backport this.

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

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#614
closes pythonGH-136852.
(cherry picked from commit aec7f5f)

Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
@bedevere-app
Copy link

bedevere-app bot commented Jul 21, 2025

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

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Jul 21, 2025
@hoodmane
Copy link
Contributor Author

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#614
closes GH-136852.
(cherry picked from commit aec7f5f)

Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
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.

Emscripten buildbot should run against node 24
3 participants