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

Simplify main WASI repo #451

Merged
merged 20 commits into from
Feb 2, 2022
Merged

Simplify main WASI repo #451

merged 20 commits into from
Feb 2, 2022

Conversation

linclark
Copy link
Member

As discussed before in meetings, we've received feedback that the current structure of the WASI repo is confusing.

This PR significantly reduces the complexity of the repo structure. It does this by moving parts of the repo that will be obsoleted to a legacy directory, and editing and consolidating the rest.

It also introduces the idea of a proposal template, which I've created a first draft of.

This version of the restructure depends on #450.

@sbc100
Copy link
Member

sbc100 commented Oct 20, 2021

Isn't wasi-libc still based on tools/witx and the files in phases/snapshot. I guess tools/witx is moving somewhere? And the witx files it processes are going to come from other repos? I think it might be good to show wasi-libc using whatever the new things is before we delete the old thing here.

@linclark
Copy link
Member Author

I guess tools/witx is moving somewhere?

Alex has been working on a tool to replace tools/witx. This is the tool that supports the new syntax which he's been demonstrating in meetings over the past couple of months. The original tools/witx will move to the legacy directory.

And the witx files it processes are going to come from other repos?

Correct. This is a change that has been in the works since late last year with the push towards modularization.

I think it might be good to show wasi-libc using whatever the new things is before we delete the old thing here.

This makes sense to me.

legacy/README.md Outdated Show resolved Hide resolved
@lygstate
Copy link
Contributor

lygstate commented Nov 6, 2021

I guess tools/witx is moving somewhere?

Alex has been working on a tool to replace tools/witx. This is the tool that supports the new syntax which he's been demonstrating in meetings over the past couple of months. The original tools/witx will move to the legacy directory.

Where is the tool, when the tool to be merged?

And the witx files it processes are going to come from other repos?

Correct. This is a change that has been in the works since late last year with the push towards modularization.

I think it might be good to show wasi-libc using whatever the new things is before we delete the old thing here.

This makes sense to me.

@linclark
Copy link
Member Author

linclark commented Nov 9, 2021

Where is the tool, when the tool to be merged?

With this simplification, the tooling would live outside of this repo, in the wasi-tools repo (which Dan just created). Once that's done, the main WASI repo would focus on coordinating work at a higher level and helping people navigate to the various APIs that are a part of WASI.

Alex is prioritizing getting the replacement tooling up in that repo, so I expect it to be available there soon.

@linclark
Copy link
Member Author

We won't be able to make a clean break with witx yet, so I updated this PR to leave the phases and tools directories in place. They will move to legacy once wit supports all APIs.

I'd like to merge this soon so that we're pointing new contributors in the right direction, so I'd ask that any word-smithing or nits be tackled in follow up PRs and that we focus comments on this PR on more substantial, structural concerns.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants