Skip to content

Provide pre-compiled shared libraries for C-API#311

Merged
tschneidereit merged 2 commits intobytecodealliance:masterfrom
yurydelendik:wasmtime-api-build
Aug 30, 2019
Merged

Provide pre-compiled shared libraries for C-API#311
tschneidereit merged 2 commits intobytecodealliance:masterfrom
yurydelendik:wasmtime-api-build

Conversation

@yurydelendik
Copy link
Contributor

Fixes #309

Copy link
Member

@tschneidereit tschneidereit left a comment

Choose a reason for hiding this comment

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

@yurydelendik thank you for doing this! \o/

It all looks great, except for two things:

  1. can you split these out into their own archives per platform? Containing the license, README, header file, and shared libraries.
  2. Please call the archives, binaries, and headers something like libwasmtime-c-api.

@yurydelendik yurydelendik force-pushed the wasmtime-api-build branch 3 times, most recently from cca1bd7 to 4626282 Compare August 30, 2019 12:29
Copy link
Member

@tschneidereit tschneidereit left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@tschneidereit tschneidereit merged commit 33e282c into bytecodealliance:master Aug 30, 2019
avanhatt pushed a commit to wellesley-prog-sys/wasmtime that referenced this pull request Apr 9, 2025
Run a subset of Cranelift and Wasmtime tests in CI to mitigate problems
such as bytecodealliance#310.

Fixes bytecodealliance#311
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