Skip to content

[config] Remove support for legacy environment variables#27381

Merged
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:remove_old_env_var_support
Jul 20, 2026
Merged

[config] Remove support for legacy environment variables#27381
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:remove_old_env_var_support

Conversation

@sbc100

@sbc100 sbc100 commented Jul 19, 2026

Copy link
Copy Markdown
Collaborator

Remove support for legacy environment variables (LLVM, BINARYEN, NODE, LLVM_ADD_VERSION, and CLANG_ADD_VERSION) in tools/config.py.

These have been generating warnings for over 5 years now (See #13922) to aid in transitioning to the modern EM_* prefixed variables (EM_LLVM_ROOT, EM_BINARYEN_ROOT, EM_NODE_JS, etc.).

Remove support for legacy environment variables (`LLVM`, `BINARYEN`,
`NODE`, `LLVM_ADD_VERSION`, and `CLANG_ADD_VERSION`) in `tools/config.py`.

These have been generating warnings for over 5 years now (See emscripten-core#13922)
to aid in transitioning to the modern `EM_*` prefixed variables
(`EM_LLVM_ROOT`, `EM_BINARYEN_ROOT`, `EM_NODE_JS`, etc.).
@sbc100
sbc100 requested review from kripken and tlively July 19, 2026 18:50
@sbc100
sbc100 enabled auto-merge (squash) July 19, 2026 22:32
@sbc100
sbc100 merged commit be4262a into emscripten-core:main Jul 20, 2026
39 checks passed
@sbc100
sbc100 deleted the remove_old_env_var_support branch July 20, 2026 14:16
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