Enable ZJIT on the available version#519
Enable ZJIT on the available version#519moznion wants to merge 2 commits intodocker-library:masterfrom
Conversation
Signed-off-by: moznion <moznion@mail.moznion.net>
Signed-off-by: moznion <moznion@mail.moznion.net>
|
Oops sorry, this seems like duplicated by #509 but is it still progress? |
|
Yeah, I'm still on it. I want to wait until the first release candiate to see how the developers feel about the status of zjit. I don't really want to expose here it if they don't think it's ready yet. For now the release notes have this to say:
|
|
Thank you (both) for the contribution! I'm indeed going to close in favor of #509 for tracking this, since @Earlopain has been really on top of it and waiting seems sane to me too ❤️ |
|
Thank you for the information. I understand the current situation. |
|
Anyway, I appreciate your support and maintenance of this repository ❤️ |
|
With the important echoed caveat that upstream doesn't officially recommend it for production yet, you should be able to create a build directly from #509 with something like this:
|
This pull request adds the
--enable-zjitoption to eachconfigureinvocation to enable ZJIT.Example after this patch is applied:
And this also upgrades the rustc version because the ZJIT runtime requires
edition2024.Note: On versions that do not support ZJIT, the
--enable-zjitoption is ignored (ref: https://github.com/ruby/docker-images/blob/7589c4dad2d112bcfc255324363c91616741261a/install_ruby.sh#L99).