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

Added support for wasm32-wasi target #119

Merged
merged 1 commit into from
Feb 1, 2022

Conversation

OtaK
Copy link
Contributor

@OtaK OtaK commented Jan 24, 2022

Hi!

I'm currently on my way to make SQLite work with wasm32-wasi.
So, one of the blockers was OpenSSL in general since SQLite depends heavily on it.

Here's my shot at it:

  • Introduces no new dependencies
  • Small alteration of the docker test harness to:
    • install wasmtime & set it up as test runner
    • install & setup wasi-sdk
  • Compatible with all versions of OpenSSL (1.1.x and 3.x.x were tested).

This should be the first step in closing sfackler/rust-openssl#1016

Should also supersede #113

Please give me any feedback! I'll tend to it ASAP.

@OtaK OtaK force-pushed the otak/wasi-compat branch from 1f937b0 to c1c5eaf Compare January 24, 2022 18:59
testcrate/src/lib.rs Outdated Show resolved Hide resolved
.github/workflows/main.yml Outdated Show resolved Hide resolved
ci/run.sh Outdated Show resolved Hide resolved
src/lib.rs Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
@alexcrichton
Copy link
Owner

Ah ok sorry I didn't realize that this wasn't passing CI, I'll take another look when CI is passing for wasm32-wasi

@OtaK OtaK force-pushed the otak/wasi-compat branch 3 times, most recently from 5446987 to 9c9b745 Compare January 30, 2022 04:38
ci/docker/wasm32-wasi/Dockerfile Outdated Show resolved Hide resolved
src/lib.rs Show resolved Hide resolved
ci/docker/wasm32-wasi/Dockerfile Outdated Show resolved Hide resolved
ci/docker/wasm32-wasi/Dockerfile Outdated Show resolved Hide resolved
@OtaK OtaK force-pushed the otak/wasi-compat branch 3 times, most recently from 23a235a to f6b8806 Compare January 31, 2022 18:16
@OtaK OtaK force-pushed the otak/wasi-compat branch from f6b8806 to 12a81c4 Compare February 1, 2022 11:01
@OtaK OtaK force-pushed the otak/wasi-compat branch from 12a81c4 to a36aee6 Compare February 1, 2022 11:08
@alexcrichton
Copy link
Owner

Looks good to me, thanks!

@alexcrichton alexcrichton merged commit 85ef012 into alexcrichton:master Feb 1, 2022
@alexcrichton alexcrichton mentioned this pull request Oct 25, 2023
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.

3 participants