Skip to content

Plumb coredump feature to wasmtime-runtime#8252

Merged
fitzgen merged 2 commits intobytecodealliance:mainfrom
alexcrichton:plumb-wasmtime-runtime-coredump
Mar 28, 2024
Merged

Plumb coredump feature to wasmtime-runtime#8252
fitzgen merged 2 commits intobytecodealliance:mainfrom
alexcrichton:plumb-wasmtime-runtime-coredump

Conversation

@alexcrichton
Copy link
Member

The wasmtime crate already has a coredump feature but whether or not it's enabled the wasmtime-runtime crate still captures a core dump. Use this flag in the wasmtime crate to plumb support to wasmtime-runtime to skip capture if it's not enabled.

The `wasmtime` crate already has a `coredump` feature but whether or not
it's enabled the `wasmtime-runtime` crate still captures a core dump.
Use this flag in the `wasmtime` crate to plumb support to
`wasmtime-runtime` to skip capture if it's not enabled.
@alexcrichton alexcrichton requested a review from a team as a code owner March 28, 2024 02:06
@alexcrichton alexcrichton requested review from fitzgen and removed request for a team March 28, 2024 02:06
@github-actions github-actions bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Mar 28, 2024
@github-actions
Copy link

Subscribe to Label Action

cc @peterhuene

Details This issue or pull request has been labeled: "wasmtime:api"

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

  • peterhuene: wasmtime:api

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

Learn more.

@fitzgen fitzgen added this pull request to the merge queue Mar 28, 2024
Merged via the queue into bytecodealliance:main with commit c6bec9c Mar 28, 2024
@alexcrichton alexcrichton deleted the plumb-wasmtime-runtime-coredump branch March 28, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasmtime:api Related to the API of the `wasmtime` crate itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants