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

Make jit a default feature for wasmer-wasm-c-api #1955

Merged
merged 3 commits into from
Dec 18, 2020

Conversation

MarkMcCaskey
Copy link
Contributor

@MarkMcCaskey MarkMcCaskey commented Dec 18, 2020

This makes cargo build now work with default features. It should also stop causing the header files to change when running tests.

Review

  • Add a short description of the the change to the CHANGELOG.md file

@MarkMcCaskey MarkMcCaskey added the 📦 lib-c-api About wasmer-c-api label Dec 18, 2020
@MarkMcCaskey
Copy link
Contributor Author

bors r+

bors bot added a commit that referenced this pull request Dec 18, 2020
1955: Make `jit` a default feature for `wasmer-wasm-c-api` r=MarkMcCaskey a=MarkMcCaskey

This makes `cargo build` now work with default features. It should also stop causing the header files to change when running tests.

# Review

- [x] Add a short description of the the change to the CHANGELOG.md file


Co-authored-by: Mark McCaskey <[email protected]>
Co-authored-by: Mark McCaskey <[email protected]>
@bors
Copy link
Contributor

bors bot commented Dec 18, 2020

This PR was included in a batch that successfully built, but then failed to merge into master (it was a non-fast-forward update). It will be automatically retried.

@bors
Copy link
Contributor

bors bot commented Dec 18, 2020

@bors bors bot merged commit 3d3b515 into master Dec 18, 2020
@bors bors bot deleted the fetaure/wasm-c-api-jit-by-default branch December 18, 2020 21:50
bors bot added a commit that referenced this pull request Dec 18, 2020
1957: Assert that no files changed during lint r=MarkMcCaskey a=MarkMcCaskey

waiting for #1955 to land, then we can merge.  The first run should fail because of the header file, then we can fix that and it should pass.  If that happens, then everything is working as expected.

# Review

- [ ] Add a short description of the the change to the CHANGELOG.md file


Co-authored-by: Mark McCaskey <[email protected]>
Co-authored-by: Mark McCaskey <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 lib-c-api About wasmer-c-api
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants