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

Reorganize WASI 0.2.0 bindings to facilitate reuse #175

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

tschneidereit
Copy link
Member

This patch doesn't introduce any functional changes: it's purely a change to how the WASI 0.2.0 host API implementation is organized. This will be used in further patches introducing more refactorings in service of more easily building WASI 0.2.x bindings making use of new WASI features.

Note that there's one part here that breaks embeddings providing their own host API: the host_api.cmake file has moved into the API implementation directory to enable more flexible organization of the implementation.

This patch doesn't introduce any functional changes: it's purely a change to how the WASI 0.2.0 host API implementation is organized. This will be used in further patches introducing more refactorings in service of more easily building WASI 0.2.x bindings making use of new WASI features.

Note that there's one part here that breaks embeddings providing their own host API: the `host_api.cmake` file has moved into the API implementation directory to enable more flexible organization of the implementation.

Signed-off-by: Till Schneidereit <till@tillschneidereit.net>
Copy link
Contributor

@guybedford guybedford left a comment

Choose a reason for hiding this comment

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

So new hosts would just be expected to include their own host_api.cmake file, but that file could just be potentially empty right?

@tschneidereit
Copy link
Member Author

So new hosts would just be expected to include their own host_api.cmake file, but that file could just be potentially empty right?

At least for now, the host_api.cmake file would need to be duplicated, though it could in principle just include another host API implementation's file. I'm open to ideas for how to reduce duplication further, though in practice I think much of what's in there will have to be slightly tweaked.

@tschneidereit tschneidereit merged commit 8dc4801 into main Nov 6, 2024
5 checks passed
@tschneidereit tschneidereit deleted the reusable-wasi-0.2.0 branch November 6, 2024 14:25
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.

2 participants