Stream: git-wasmtime

Topic: wasmtime / PR #14779 write the coredump to the configured...


view this post on Zulip Wasmtime GitHub notifications bot (Oct 16 2026 at 14:39):

xia-chao opened PR #14779 from xia-chao:serve-coredump-file to bytecodealliance:main:

run writes the coredump with -D coredump=<path>; serve previously only printed it to stderr.
Now, for the same option, serve writes it too.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 16 2026 at 14:39):

xia-chao requested dicej for a review on PR #14779.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 16 2026 at 14:39):

xia-chao requested wasmtime-core-reviewers for a review on PR #14779.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 16 2026 at 14:52):

:memo: alexcrichton submitted PR review:

It looks like this is all a dupe of the handling code in run.rs I think -- would it be possible to perhaps deduplicate that to a shared common implementation?

view this post on Zulip Wasmtime GitHub notifications bot (Oct 16 2026 at 15:33):

xia-chao updated PR #14779.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 16 2026 at 15:42):

xia-chao commented on PR #14779:

Not entirely, because this is a scam for handling code in run.rs.
Thank you, it's already gone heavy!

view this post on Zulip Wasmtime GitHub notifications bot (Oct 16 2026 at 15:44):

xia-chao deleted a comment on PR #14779:

Not entirely, because this is a scam for handling code in run.rs.
Thank you, it's already gone heavy!

view this post on Zulip Wasmtime GitHub notifications bot (Oct 16 2026 at 15:48):

xia-chao commented on PR #14779:

not entirely. the write was a copy so I shared it. the checks aren't the same.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 15:02):

:thumbs_up: alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 15:02):

alexcrichton added PR #14779 write the coredump to the configured path to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 16:41):

:check: alexcrichton merged PR #14779.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 16:41):

alexcrichton removed PR #14779 write the coredump to the configured path from the merge queue.


Last updated: Oct 31 2026 at 23:03 UTC