Skip to content
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

Bump emscripten to 3.1.30 #282

Merged
merged 51 commits into from
Feb 7, 2023
Merged

Conversation

radekdoulik
Copy link
Member

No description provided.

kjlubick and others added 30 commits April 27, 2022 13:13
…pten-core#1037)

* [bazel] Set CLOSURE_COMPILER to workaround RBE+symlinks issue

* space

* specify node_js
* Optimize sandbox performance

Link just the files needed to compile, link, or archive, rather than the entire directory tree. This drastically improves build times on macOS, where the sandbox is known to be slow (bazelbuild/bazel#8230).

* Linux wants clang to link

* all_files not needed?

* Linux wants llc

* And llvm-ar

* Templated build_file_content
* Explicit outputs for wasm_cc_binary

* Backwards compatibility

* data_runfiles restore

* restore test_bazel.sh

* Using wrong path on accident

* two separate rules for legacy support

* Added name attribute to wasm_cc_binary rule
* 3.1.18

* Update LLVM include path in Bazel files
3.1.18 had a bad release binary on ARM64 Mac so push an updated version of the release.
Without this the recommended way to silence emsdk_env was to pipe its
stderr to /dev/null.. but then you also loose potentially useful error
message.

Fixes: dotnet#946
Newer versions of emscipten, starting all the way back in 1.39.13, can
automatically locate the `.emscripten` config file that emsdk creates so
there is no need for the explicit EM_CONFIG environment variable.  Its
redundant and adds unnessary noisce/complexity.

Really, adding emcc to the PATH is all the is needed these days.

One nice thing about this change is that it allows folks to run
whichever emcc they want to and have it just work, even if they have
configured emsdk.   Without this change, if I activate emsdk and I run
`some/other/emcc` then emsdk's `EM_CONFIG` will still be present and
override the configuration embedded in `some/other/emcc`.

e.g. in the same shell, with emsdk activated, I can run both these
commands and have them both just work as expected.

```
$ emcc --version
$ /path/to/my/emcc --version
```
eng/emsdk.proj Outdated Show resolved Hide resolved
radekdoulik and others added 2 commits January 30, 2023 18:26
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
@lewing lewing merged commit f55ecf5 into dotnet:main Feb 7, 2023
radical added a commit to radical/emsdk that referenced this pull request Feb 8, 2023
radical added a commit that referenced this pull request Feb 8, 2023
radekdoulik added a commit to radekdoulik/emsdk that referenced this pull request Feb 22, 2023
lewing pushed a commit that referenced this pull request Feb 22, 2023
* Revert "Revert "Bump emscripten to 3.1.30 (#282)" (#291)"

This reverts commit a117e26.

* Also update clang version in the wrapper
radekdoulik added a commit to radekdoulik/emsdk that referenced this pull request Feb 28, 2023
* Revert "Revert "Bump emscripten to 3.1.30 (dotnet#282)" (dotnet#291)"

This reverts commit a117e26.

* Also update clang version in the wrapper
radekdoulik added a commit to radekdoulik/emsdk that referenced this pull request Mar 9, 2023
* Revert "Revert "Bump emscripten to 3.1.30 (dotnet#282)" (dotnet#291)"

This reverts commit a117e26.

* Also update clang version in the wrapper
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.