Skip to content

Conversation

@loganek
Copy link
Collaborator

@loganek loganek commented Jan 19, 2024

When WAMR is embedded to other application, the lifecycle of the socket might conflict with other usecases. E.g. if WAMR is deinitialized before any other use of sockets, the application goes into the invalid state. The new flag allows host application to take control over the socket initialization.

When WAMR is embedded to other application, the lifecycle of the socket might
conflict with other usecases. E.g. if WAMR is deinitialized before any other
use of sockets, the application goes into the invalid state. The new flag
allows host application to take control over the socket initialization.
@loganek
Copy link
Collaborator Author

loganek commented Jan 21, 2024

@wenyongh does the change look good to you? If so, do you think we can merge it?

@wenyongh wenyongh merged commit 627debc into bytecodealliance:main Jan 21, 2024
@wenyongh
Copy link
Contributor

@wenyongh does the change look good to you? If so, do you think we can merge it?

Sure, I merged it.

vickiegpt pushed a commit to vickiegpt/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
When WAMR is embedded to other application, the lifecycle of the socket
might conflict with other usecases. E.g. if WAMR is deinitialized before any
other use of sockets, the application goes into the invalid state. The new
flag allows host application to take control over the socket initialization.
@loganek loganek deleted the loganek/optional-sock-init branch June 10, 2024 12:47
g0djan pushed a commit to g0djan/wasm-micro-runtime that referenced this pull request Sep 30, 2024
When WAMR is embedded to other application, the lifecycle of the socket
might conflict with other usecases. E.g. if WAMR is deinitialized before any
other use of sockets, the application goes into the invalid state. The new
flag allows host application to take control over the socket initialization.
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