Skip to content

Add option to skip building shared library#322

Merged
aduh95 merged 1 commit into
nodejs:mainfrom
vshymanskyy:patch-01
Jul 3, 2026
Merged

Add option to skip building shared library#322
aduh95 merged 1 commit into
nodejs:mainfrom
vshymanskyy:patch-01

Conversation

@vshymanskyy

Copy link
Copy Markdown
Contributor

Added UVWASI_BUILD_SHARED option, similar to that of libuv (LIBUV_BUILD_SHARED).
I needed to disable the shared library build for some target platforms of Wasm3, otherwise the build fails.

@vshymanskyy

Copy link
Copy Markdown
Contributor Author

@aduh95 this is the second part of my changes.
thank you

@aduh95 aduh95 merged commit 0820128 into nodejs:main Jul 3, 2026
8 checks passed
@aduh95

aduh95 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

I might have merged too soon, I don't seem to be able to build when I pass -DUVWASI_BUILD_SHARED:BOOL=FALSE:

CMake Error at CMakeLists.txt:168 (install):
  install TARGETS given target "uvwasi" which does not exist.

@vshymanskyy

Copy link
Copy Markdown
Contributor Author

Oh, weird. Will recheck tomorrow

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