Skip to content

Fix ruby-head-wasm-wasi builds on Ruby head with host dump_ast#630

Merged
kateinoigakukun merged 1 commit intoruby:mainfrom
ledsun:fix-issue-629-dump-ast
Mar 22, 2026
Merged

Fix ruby-head-wasm-wasi builds on Ruby head with host dump_ast#630
kateinoigakukun merged 1 commit intoruby:mainfrom
ledsun:fix-issue-629-dump-ast

Conversation

@ledsun
Copy link
Contributor

@ledsun ledsun commented Mar 22, 2026

Summary

  • pass the host-built dump_ast to cross builds alongside --with-baseruby
  • avoid running the target-side wasm dump_ast during .rbinc generation on current Ruby head
  • declare dump_ast_path in RBS and document why the configure argument is needed

Testing

  • bundle exec rake check:type

Closes #629.

Pass the host-built dump_ast alongside --with-baseruby so cross builds do not try to execute the target-side wasm dump_ast while generating .rbinc files on current Ruby head.

Also declare dump_ast_path in RBS and document why the configure argument is needed.
@ledsun ledsun force-pushed the fix-issue-629-dump-ast branch from c6edc92 to e4a524c Compare March 22, 2026 11:48
@ledsun ledsun marked this pull request as ready for review March 22, 2026 12:21
Copy link
Member

@kateinoigakukun kateinoigakukun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🙏

@kateinoigakukun kateinoigakukun merged commit 728975b into ruby:main Mar 22, 2026
38 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.

ruby-head-wasm-wasi build fails on current Ruby head because cross build runs wasm dump_ast instead of host dump_ast

2 participants