-
Notifications
You must be signed in to change notification settings - Fork 93
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
Update to libv8-node 17.x #232
Conversation
3a42551
to
53d0cbe
Compare
This also requires to bump the version requirement for libv8-node (once a new version is released), right? |
@tisba I believe I did do so? The version requirement for libv8-node in the gemspec references the |
@nightpool any news on the libv8 upgrade? nodejs 18 is already out it appears |
Currently building the libv8-node 17.3.1 packages. Testing against this PR produces a missing language tag when testing locales:
|
It gets better, if I replace |
Here's the output:
|
Let me know if you need help testing on Apple Silicon M1. |
@tisba thanks, I have both Intel and M1 machines at hand, so that's fine :) |
Perfect! Let me know if I can help in any way. I'll wait then and run it through the test suite of projects I'm involved with that make heavy use of mini_racer. |
I have also started a |
Hmm, in CI it seems like cross-compiling does not produce a segfault (the one that segfaults is locally built and tested natively on aarch64) but native does. |
Oh, and I bumped to the most recent 17.9.1 on the |
@@ -2,5 +2,5 @@ | |||
|
|||
module MiniRacer | |||
VERSION = "0.6.2" | |||
LIBV8_NODE_VERSION = "~> 16.10.0.0" | |||
LIBV8_NODE_VERSION = "~> 17.3.1.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LIBV8_NODE_VERSION = "~> 17.3.1.0" | |
LIBV8_NODE_VERSION = "~> 17.9.1.0" |
|
PR for node 18: #261 Note that even if that one above works, we may still want a node 17 based release of mini_racer as the requirements for node 18 have increased a lot from node 17 (e.g macOS 10.13 -> macOS 10.15), so we should probably fix this one and merge it first. |
I'm going to publish these new libv8-node 16+17+18 builds even though #259 is a thing, I think it will ease everyone's life testing stuff that's not reliant on locale. |
yikes the segfault is certainly concerning, but I agree getting these published will help a ton. |
16.17.0, 17.9.1, 18.8.0 gems pushed for all platforms. Note that the most recent rubygems now supports I did not make use of that yet, but will in the future. Being able to specify |
Closing as it was superseded by #271 |
Depends on rubyjs/libv8-node#31