-
Notifications
You must be signed in to change notification settings - Fork 188
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
TravisCI caching and Bundler use the same path for C extensions for different TruffleRuby versions #1715
Comments
From that log I see that paths like these are used:
|
I can reproduce the problem locally, and confirm setting |
* See #1715 * Fix path to bundled gems inside the TruffleRuby home.
Fixed by 5ed7adc, at least for the Bundler case.
|
Let's track the fix for chruby separately: #1723, and close this issue since it will be fixed in the upcoming release. |
TruffleRuby 19.2.0 was released and fixes this bug, please try it. |
This leads to errors such as:
Example build: https://travis-ci.org/socketry/async-websocket/jobs/554414769
We see
msgpack
is not recompiled (Using msgpack 1.3.0
in the log), even though it was compiled with TruffleRuby 19.0.0 and now reused on TruffleRuby 19.1.0, which doesn't work.TravisCI caching seems to simply ignore the TruffleRuby version when using:
The cache URLs seem to just consider the value of
rvm:
:This seems like a bug that needs to be fixed in TravisCI.
If
RUBY_ENGINE_VERSION
was used, then it should work out of the box.A workaround until this is fixed in the next release is to manually delete the caches on TravisCI.
This is necessary only if having a cache against 19.0.0 and needs to be done only once, as after the cache will be rebuilt against 19.1.0 and the next release will include a fix this.
On the TruffleRuby side, maybe we could try to have a better error in such a case, but that's probably quite difficult to achieve, because we'd need a check before starting to load the C extension.
cc @ioquatix
The text was updated successfully, but these errors were encountered: