Stream: git-wasmtime

Topic: wasmtime / PR #7510 wasi-filesystem: delete access modes


view this post on Zulip Wasmtime GitHub notifications bot (Nov 08 2023 at 23:26):

pchickey opened PR #7510 from bytecodealliance:pch/delete_filesystem_modes to bytecodealliance:main.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 08 2023 at 23:33):

pchickey updated PR #7510.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 08 2023 at 23:38):

pchickey edited PR #7510.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 08 2023 at 23:38):

pchickey has marked PR #7510 as ready for review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 08 2023 at 23:38):

pchickey requested wasmtime-core-reviewers for a review on PR #7510.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 08 2023 at 23:38):

pchickey requested alexcrichton for a review on PR #7510.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 14:48):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 14:48):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 14:48):

alexcrichton created PR review comment:

Question on this, if we know we want to add this in the future should we be trying to future-proof this function? For example we could add an options: option<open-options> with a currently empty resource open-options or something like that.

Alternatively we could add a feature to the component model and WIT for something like keyword arguments with defaults. That's a much longer- term vision though.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 17:57):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 17:57):

pchickey created PR review comment:

I believe that we can eliminate the modes parameter here and let open-at continue to do the reasonable default behavior, and then when users of the re-introduced modes stuff want to opt into a non-default behavior for modes, we can expose a new open-at-modes function that has the additional argument. Then when it comes time to make a breaking change in preview 3 we can consolidate down to just one open-at with mode arguments.

We could get fancier than that, but I expect the above will work fine without introducing much complexity.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 19:10):

alexcrichton merged PR #7510.


Last updated: Dec 23 2024 at 12:05 UTC