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

Build WASM with emcc 2.x #429

Merged
merged 10 commits into from
May 24, 2022
Merged

Build WASM with emcc 2.x #429

merged 10 commits into from
May 24, 2022

Conversation

fundthmcalculus
Copy link
Contributor

https://users.rust-lang.org/t/wasm-unknown-vs-emscripten/22997/5
We should be using unknown over emscripten

We should be using `unknown` over `emscripten`
@github-actions
Copy link
Contributor

github-actions bot commented May 20, 2022

.NET Test Report

0 files  ±0  0 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 092efea. ± Comparison against base commit 924db1d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented May 20, 2022

Golang Test Report

  3 files  ±0    3 suites  ±0   0s ⏱️ -1s
16 tests ±0  16 ✔️ ±0  0 💤 ±0  0 ±0 
48 runs  ±0  48 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 092efea. ± Comparison against base commit 924db1d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented May 20, 2022

Python Test Report

  12 files  ±0    12 suites  ±0   22s ⏱️ +9s
  14 tests ±0    14 ✔️ ±0  0 💤 ±0  0 ±0 
168 runs  ±0  168 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 092efea. ± Comparison against base commit 924db1d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented May 20, 2022

Ruby Test Report

  60 files  ±0    60 suites  ±0   3s ⏱️ ±0s
  14 tests ±0    14 ✔️ ±0  0 💤 ±0  0 ±0 
168 runs  ±0  168 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 092efea. ± Comparison against base commit 924db1d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented May 20, 2022

Java Test Report

12 files  ±0  12 suites  ±0   2s ⏱️ ±0s
19 tests ±0  19 ✔️ ±0  0 💤 ±0  0 ±0 
45 runs  ±0  45 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 092efea. ± Comparison against base commit 924db1d.

This pull request removes 2 and adds 2 tests. Note that renamed tests count towards both.
OkapiNativeTest ‑ [1] OkapiNativeTest$DataArgument@2ef14fe
OkapiNativeTest ‑ [2] OkapiNativeTest$DataArgument@53de625d
OkapiNativeTest ‑ [1] OkapiNativeTest$DataArgument@49ff7d8c
OkapiNativeTest ‑ [2] OkapiNativeTest$DataArgument@648c94da

♻️ This comment has been updated with latest results.

@fundthmcalculus fundthmcalculus enabled auto-merge (squash) May 20, 2022 16:41
Copy link
Member

@tmarkovski tmarkovski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine adding unknown-unknown as additional wasm output, though right now, we have no need for it. emscripten is used by the blazor linker, which uses emcc which needs -emscripten target

@fundthmcalculus
Copy link
Contributor Author

I'm fine adding unknown-unknown as additional wasm output, though right now, we have no need for it. emscripten is used by the blazor linker, which uses emcc which needs -emscripten target

Copy. I will try to make oberon compile with emscripten then.

@tmarkovski tmarkovski disabled auto-merge May 22, 2022 17:57
@fundthmcalculus fundthmcalculus changed the title Switch WASM target to properly build Build WASM with emcc 2.x May 23, 2022
@fundthmcalculus fundthmcalculus enabled auto-merge (squash) May 23, 2022 20:56
@fundthmcalculus fundthmcalculus merged commit 7009a18 into main May 24, 2022
@fundthmcalculus fundthmcalculus deleted the fix-wasm-emscriptem branch May 24, 2022 11:55
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