-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add support for Wasm #95
Comments
I would love that too if it is possible. |
Even if it is just annotations, so we can annotate classes in |
@sgammon can you please elaborate on your comment? What annotations are you talking about? |
@Mr3zee Sorry, I mean annotations or interfaces. Things that can safely be included in a common target, so that we can at least create our RPC interfaces in a common target, even if we can't implement them there yet. |
I see, thank you |
@Mr3zee when can we plan with wasm support? |
If I understand correctly, this feature will be delivered in the next version of kotlinx-rpc. Do you have any idea when ? |
No ETA for now |
@Mr3zee Could a WASM target be added which is functionally empty? Or even just has Because KotlinX RPC does not ship a WASM target, we cannot use KotlinX RPC in any source set which ships for WASM, or which ultimately is depended on downstream by a source set for WASM, whether or not those source sets make use of RPC. |
I would humbly ask that you reconsider because KotlinX RPC is the only KotlinX library we cannot use due to this reason |
I very agree, just to compile projet and permit to have a shared module (multiplatform one) in projects which have the goal to target WASM in the future. It can be useful to juste have an empty dependency compliant to this target. |
WASI is blocked by oshai/kotlin-logging#433, |
I would actually be surprised to find |
@sgammon It is a dependency for the kRPC protocol implementation Necessity for the logging dependency will be assessed before going stable |
@Mr3zee Am I correct in my assumption that this is the only place a KotlinX library depends on something outside of |
@sgammon technically, yes (though I'm not sure about all of the other kotlinx libs). But there is a little more to that. Our core module depends only on atomicfu, coroutines and serialization. Core module and codegen plugins make the foundation of the library, so being dependent on core libs is only logical. Then kRPC (and other modules in work) is an implementation of the library foundation, more complex than core itself, and requires external dependencies. And while logging is a debatable topic here, and it may happen, that kRPC will also be dependent only on core libs, for gRPC, for example, it surely will not be the case, as we will use existing platform specific libs to provide this integration. Implementing gRPC protocol from scratch is not what we want to do. |
@Mr3zee Thank you so much for shipping this one! |
Please deploy artifact for the
wasmJs
&wasmWasi
targets.The text was updated successfully, but these errors were encountered: