Skip to content

Conversation

@sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Nov 8, 2025

This tests was broken on systems where the python executable lives in path that also contains clang executable.

We were remove PATH elements here, but we cannot remove the one that contains the python executable otherwise the bootstrap script cannot run at all.

@sbc100 sbc100 requested a review from juj November 8, 2025 18:38
This tests was broken on systems where the python executable lives in path
that also contains clang executable.

We were remove PATH elements here, but we cannot remove the one that
contains the python executable otherwise the bootstrap script cannot run
at all.
@sbc100 sbc100 force-pushed the test_bootstrap_without_em_config branch from ec751c9 to c3f1f5e Compare November 8, 2025 18:38
@sbc100
Copy link
Collaborator Author

sbc100 commented Nov 8, 2025

This test was failing on my system, I'm not sure for how long.

@sbc100 sbc100 requested a review from kripken November 8, 2025 20:45
@sbc100 sbc100 merged commit 36d6f60 into emscripten-core:main Nov 8, 2025
33 of 34 checks passed
@sbc100 sbc100 deleted the test_bootstrap_without_em_config branch November 8, 2025 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants