-
Notifications
You must be signed in to change notification settings - Fork 116
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
Publish javy-cli
to crates.io
#792
Comments
If you're willing to have your service spawn a process (e.g., through https://doc.rust-lang.org/std/process/struct.Command.html) then publishing Generally (and assuming I'm understanding your use-case correctly) I think we should be able to do a bit better, such that instead of having to spawn a process from your application, you should be able to add a a library dependency that allows building a Wasm module from a JavaScript source. This shouldn't be too hard to accomplish if we start by extracting the code generation pieces used by the cli and publish those as a separate crate (e.g., [dependencies]
javy-codegen = 1 Note that even though the refactoring to extract this code might be trivial, some open questions still stand, like for example:
I think some exploration is needed around those areas to have a better handle of the pros/cons. |
It's a few months late but I took a swing at implementing your suggested changes. |
I just wanted to follow up on an outstanding request to publish
javy-cli
to crates as mentioned in this thread. I'm not sure my requirements are exactly the same as the original requester but my use case is having a rust service that executes compilation at runtime and stores the result somewhere to execute later. I believe the code injavy-cli
can accomplish most of what I want and I would rather consume it as a library from crates rather then fork it.Regarding the original thread it was mentioned that we should start a ticket up to discuss different approaches so I'm going ahead with that here. I'm open to helping out with the contribution but I'm interested in hearing what approaches the maintainers think are best.
The text was updated successfully, but these errors were encountered: