Skip to content

Disable wasmtime default-features in wasi crate#2664

Merged
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
maxded:disable-wasmtime-default-features-wasi
Feb 18, 2021
Merged

Disable wasmtime default-features in wasi crate#2664
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
maxded:disable-wasmtime-default-features-wasi

Conversation

@maxded
Copy link
Copy Markdown
Contributor

@maxded maxded commented Feb 18, 2021

Disabling wasmtime's default features in the wasi crate has been removed in the transition from 0.22.0 to 0.23.0. This brings in a whole lot of unnecessary features and dependencies that the wasi crate doesn't need.

Not sure if there was a specific reason for the removal or just a small accident. Anyways, here is a small change to disable wasmtime's default features again. :)

Copy link
Copy Markdown
Member

@alexcrichton alexcrichton left a comment

Choose a reason for hiding this comment

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

Oops, thanks for catching this!

@github-actions github-actions Bot added the wasi Issues pertaining to WASI label Feb 18, 2021
@github-actions
Copy link
Copy Markdown

Subscribe to Label Action

cc @kubkon

Details This issue or pull request has been labeled: "wasi"

Thus the following users have been cc'd because of the following labels:

  • kubkon: wasi

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

@pchickey
Copy link
Copy Markdown
Contributor

Sorry, my bad, this got lost in a big PR.

@alexcrichton alexcrichton merged commit 445e539 into bytecodealliance:main Feb 18, 2021
peterhuene pushed a commit to peterhuene/wasmtime that referenced this pull request Feb 19, 2021
@bnjbvr bnjbvr mentioned this pull request Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasi Issues pertaining to WASI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants