Skip to content

[PROF-13303] Add Ruby 4.0.0-preview3 headers and bump gem version to 3.4.3#22

Merged
ivoanjo merged 4 commits into
masterfrom
ivoanjo/prof-13303-update-ruby4-headers
Dec 18, 2025
Merged

[PROF-13303] Add Ruby 4.0.0-preview3 headers and bump gem version to 3.4.3#22
ivoanjo merged 4 commits into
masterfrom
ivoanjo/prof-13303-update-ruby4-headers

Conversation

@ivoanjo

@ivoanjo ivoanjo commented Dec 18, 2025

Copy link
Copy Markdown
Member

What does this PR do?

This PR imports the Ruby 4.0.0-preview3 headers into the gem.

Motivation:

The 4.0.0-preview2 headers are not compatible with preview3 -- in particular there's a new field in the thread structure that causes the offset of everything that comes after it to shift, and thus makes the profiler crash on preview3 when compiled with preview2 headers.

Additional Notes:

This PR is very similar to #15, and I followed the usual instructions from the README.md to extract the headers.

I've decided to delete the changelog file as it's easy to forget (it was already outdated), and changes in this gem/repo are easy to identify by looking at PRs/commits so this didn't offer much value.

How to test the change?

I've tested this PR by running the ruby profiler test suite on Ruby 4.0.0-preview3. Other than tiny small behavior changes that happened between preview2 and preview3 the ruby profiler test suite is green with the updated headers.

At some point we'll have CI in dd-trace-rb for Ruby 4.0.0-preview3 (or later), but currently this needs to be tested locally.

This tiny change came in handy for testing, so "why not"?
The 4.0.0-preview2 headers are not compatible with -preview3 --
in particular there's a new field in the thread structure that changes
the offset of everything that comes after it, and thus makes the
profiler crash on preview3 when compiled with preview2 headers.

I followed the usual instructions from the `README.md` to extract
the headers.
The changelog is easy to forget, and anyway changes in this gem/repo
are easy to identify by looking at PRs/commits so this doesn't offer
much value IMHO.
@ivoanjo
ivoanjo merged commit fb17c71 into master Dec 18, 2025
19 checks passed
@ivoanjo
ivoanjo deleted the ivoanjo/prof-13303-update-ruby4-headers branch December 18, 2025 17:25
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