Minor improvements to the Emscripten instructions#1795
Minor improvements to the Emscripten instructions#1795hoodmane wants to merge 6 commits intopython:mainfrom
Conversation
Documentation build overview
|
| by setting the ``EM_COMPILER_WRAPPER`` environment, but this step will only | ||
| take effect if it is done **after** ``emsdk_env.sh`` is sourced (otherwise, the |
There was a problem hiding this comment.
This would be difficult to do now since our script automatically sources emsdk_env.sh. Thankfully it is not true that it has to be set after sourcing emsdk_env.sh. Ccache must be at least version 4.3
| subsequent runs unless there is a change in the version or build script for the | ||
| dependency. | ||
|
|
||
| It is assumed that nvm_ is installed in `${HOME}/.nvm`. |
There was a problem hiding this comment.
Ideally, we should search for nvm first in NVM_DIR then in XDG_CONFIG_HOME and then in HOME since that's where nvm likes to install itself. But we don't currently do that.
There was a problem hiding this comment.
Also, we should add a HOST_RUNNER environment variable so people can set export HOST_RUNNER=node if they don't want nvm to be used. Currently you can do that by passing --host-runner=node so I guess I can document that much.
1 participant
No description provided.