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.

Sign up for GitHub

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

Jump to bottom

Convert WAVM Wasm runtime to an extension #12574

Open
jplevyak opened this issue Aug 10, 2020 · 0 comments
Open

Convert WAVM Wasm runtime to an extension #12574

jplevyak opened this issue Aug 10, 2020 · 0 comments
Labels
area/wasm help wanted Needs help!

Comments

@jplevyak
Copy link
Contributor

Currently the WAVM Wasm runtime support is a define option: --define wasm=enabled or --define wasm=wavm.

Instead we can convert the WAVM runtime support to an extension and then use a registry to resolve it. We can
do the same thing for V8 and any other Wasm runtimes.
@jplevyak jplevyak mentioned this issue Aug 10, 2020
Merged
This was referenced Nov 10, 2020
Merged
Merged
lizan added a commit that referenced this issue Nov 17, 2020
@lizan
wasm: use static registration for runtimes (#14014)
77e1372 
Partially addresses #12574. Refactored test instantiate to removes many ifdefs.

Commit Message:
Additional Description:
Risk Level: Low
Testing: CI
Docs Changes:
Release Notes:
Platform Specific Features:

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@lizan lizan mentioned this issue Nov 17, 2020
Merged
lizan added a commit that referenced this issue Nov 18, 2020
@lizan
wasm: make dependency clearer (#14062)
e72d3e5 
Partially addresses #12574, makes split the monolithic wasm_lib into multiple libraries. Clear up dependency metadata due to select.

Risk Level: Low
Testing: CI

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@lizan lizan mentioned this issue Nov 19, 2020
Closed
andreyprezotto pushed a commit to andreyprezotto/envoy that referenced this issue Nov 24, 2020
@lizan @andreyprezotto
wasm: use static registration for runtimes (envoyproxy#14014)
bbfe4fb 
Partially addresses envoyproxy#12574. Refactored test instantiate to removes many ifdefs.

Commit Message:
Additional Description:
Risk Level: Low
Testing: CI
Docs Changes:
Release Notes:
Platform Specific Features:

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
andreyprezotto pushed a commit to andreyprezotto/envoy that referenced this issue Nov 24, 2020
@lizan @andreyprezotto
wasm: make dependency clearer (envoyproxy#14062)
81b02b6 
Partially addresses envoyproxy#12574, makes split the monolithic wasm_lib into multiple libraries. Clear up dependency metadata due to select.

Risk Level: Low
Testing: CI

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
qqustc pushed a commit to qqustc/envoy that referenced this issue Nov 24, 2020
@lizan @qqustc
wasm: use static registration for runtimes (envoyproxy#14014)
ca7a6ee 
Partially addresses envoyproxy#12574. Refactored test instantiate to removes many ifdefs.

Commit Message:
Additional Description:
Risk Level: Low
Testing: CI
Docs Changes:
Release Notes:
Platform Specific Features:

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Qin Qin <qqin@google.com>
qqustc pushed a commit to qqustc/envoy that referenced this issue Nov 24, 2020
@lizan @qqustc
wasm: make dependency clearer (envoyproxy#14062)
29ccb20 
Partially addresses envoyproxy#12574, makes split the monolithic wasm_lib into multiple libraries. Clear up dependency metadata due to select.

Risk Level: Low
Testing: CI

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Qin Qin <qqin@google.com>
@PiotrSikora PiotrSikora mentioned this issue May 2, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/wasm help wanted Needs help!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jplevyak @mattklein123