Skip to content
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

javy compile should use javy_quickjs_provider_v2 #766

Closed
saulecabrera opened this issue Sep 27, 2024 · 0 comments · Fixed by #777
Closed

javy compile should use javy_quickjs_provider_v2 #766

saulecabrera opened this issue Sep 27, 2024 · 0 comments · Fixed by #777
Labels
bug Something isn't working

Comments

@saulecabrera
Copy link
Member

saulecabrera commented Sep 27, 2024

We're in the process of deprecating the CLI's compile command in favor of the build command, for more details see: #702.

As such, the compile command is considered to be frozen and is only guaranteed to be compatible with the javy_quickjs_provider_v2 provider module, therefore the compile should use the v2 provider instead of the latest available one. This not causing any issues at the moment, but it could cause potential issues in case a new version is released in which incompatible QuickJS bytecode is emitted.

The proposed solution is to use the v2 provider, available in-tree when the compile command is invoked.

@saulecabrera saulecabrera added the bug Something isn't working label Sep 27, 2024
saulecabrera added a commit to saulecabrera/javy that referenced this issue Oct 4, 2024
Closes bytecodealliance#766

This commit ensures that the soon-to-be-deprecated `compile` command
makes use of the right (v2) provider for compilation. Not ensuring this
has the potential to cause issues upon a new release of QuickJS
containing bytecode incompatibilities.
saulecabrera added a commit that referenced this issue Oct 7, 2024
* Use the v2 provider for `compile`

Closes #766

This commit ensures that the soon-to-be-deprecated `compile` command
makes use of the right (v2) provider for compilation. Not ensuring this
has the potential to cause issues upon a new release of QuickJS
containing bytecode incompatibilities.

* Rename param
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant