search

nodejs / uvwasi

WASI syscall API built atop libuv
MIT License
225 stars 49 forks source link

feat!: Allow project to be installable #210

Closed phated closed 1 year ago

phated commented 1 year ago

This reworks the cmake build of the project to add install targets for uvwasi. This will make it possible for pkg-config or cmake build pipelines to discover globally installed uvwasi.

To install uvwasi, you'd run cmake --install out after you build.

I also removed the WITH_SYSTEM_LIBUV option because we should be using the globally installed libuv if it can be found. If people are very against this change, I can introduce a flag to disable this behavior.

I've tested this against a local branch of wasm-micro-runtime and it seems to be working. My plan is to add that build to CI once they accept my changes.

Closes #199

phated commented 1 year ago

I'm definitely no expert in this stuff, so happy to take any feedback.