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

GraalVM rc7 removed method LocalizationSupport#addToCache #20

Closed
Sanne opened this issue Sep 14, 2018 · 1 comment
Closed

GraalVM rc7 removed method LocalizationSupport#addToCache #20

Sanne opened this issue Sep 14, 2018 · 1 comment
Milestone

Comments

@Sanne
Copy link
Member

Sanne commented Sep 14, 2018

We'll need to figure out a better alternative for Shamrock.

In the meantime when testing things with Graal "master" I've used this additional commit to restore compatibility:

https://github.com/protean-project/graal/commit/581706eb9aaf417646eba6188dbf2eae92d308f0

@stuartwdouglas
Copy link
Member

This should actually be pretty easy to fix once RC7 is released, it looks like the method was just renamed. If we want something that works on both then I could add some reflection based thing but I think it might be easier to just wait.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants