Skip to content

Fix truffleruby CI for 34.0.0 release#4076

Merged
kddnewton merged 1 commit intoruby:mainfrom
Earlopain:fix-truffleruby
Apr 13, 2026
Merged

Fix truffleruby CI for 34.0.0 release#4076
kddnewton merged 1 commit intoruby:mainfrom
Earlopain:fix-truffleruby

Conversation

@Earlopain
Copy link
Copy Markdown
Collaborator

Because truffleruby now uses prism itself for its ripper implementation, the templates need to be created beforehand.

So, move the ripper require inside the tasks itself, after which the templates have been created.

Example failure: https://github.com/ruby/prism/actions/runs/24358711087/job/71132405299

LoadError: cannot load such file -- /home/runner/work/prism/prism/lib/prism/node (LoadError)
/home/runner/work/prism/prism/lib/prism.rb:127:in '<top (required)>'
/home/runner/work/prism/prism/rakelib/lex.rake:4:in '<top (required)>'
/home/runner/work/prism/prism/vendor/bundle/truffleruby/34.0.0.1/gems/rake-13.3.1/exe/rake:27:in '<top (required)>'
/home/runner/.rubies/truffleruby-34.0.0/bin/bundle:25:in '<main>'
(See full trace by running task with --trace)

lex.rake:4 is require "ripper"

…on, the templates need to be created beforehand.

So, move the ripper require inside the tasks itself, after which the templates have been created.

Example failure: https://github.com/ruby/prism/actions/runs/24358711087/job/71132405299

```
LoadError: cannot load such file -- /home/runner/work/prism/prism/lib/prism/node (LoadError)
/home/runner/work/prism/prism/lib/prism.rb:127:in '<top (required)>'
/home/runner/work/prism/prism/rakelib/lex.rake:4:in '<top (required)>'
/home/runner/work/prism/prism/vendor/bundle/truffleruby/34.0.0.1/gems/rake-13.3.1/exe/rake:27:in '<top (required)>'
/home/runner/.rubies/truffleruby-34.0.0/bin/bundle:25:in '<main>'
(See full trace by running task with --trace)
```

`lex.rake:4` is `require "ripper"`
@kddnewton kddnewton merged commit 8ab2901 into ruby:main Apr 13, 2026
68 checks passed
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

Successfully merging this pull request may close these issues.

2 participants