Skip to content

Upgrade CppInterOp to master#21961

Merged
aaronj0 merged 1 commit into
root-project:masterfrom
aaronj0:cppinterop-upgrade
Apr 23, 2026
Merged

Upgrade CppInterOp to master#21961
aaronj0 merged 1 commit into
root-project:masterfrom
aaronj0:cppinterop-upgrade

Conversation

@aaronj0

@aaronj0 aaronj0 commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Brings in fixes for GetType, support for tracing crashes, per-interpreter JitCall wrapper cache, improved handling of the interpreter creation and destruction

@aaronj0 aaronj0 self-assigned this Apr 20, 2026
@github-actions

github-actions Bot commented Apr 20, 2026

Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 8h 18m 10s ⏱️
 3 850 tests  3 848 ✅  1 💤 1 ❌
76 007 runs  75 988 ✅ 18 💤 1 ❌

For more details on these failures, see this check.

Results for commit aa70d49.

♻️ This comment has been updated with latest results.

@aaronj0 aaronj0 force-pushed the cppinterop-upgrade branch 7 times, most recently from ae13591 to fedb695 Compare April 23, 2026 08:38
@aaronj0 aaronj0 force-pushed the cppinterop-upgrade branch from fedb695 to aa70d49 Compare April 23, 2026 08:39
@aaronj0 aaronj0 requested a review from guitargeek April 23, 2026 11:22
@aaronj0

aaronj0 commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author

Test failure on alma10 looks unrelated. The windows CI seems to run out of space
cc @guitargeek @bellenot

@guitargeek guitargeek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks!

@aaronj0

aaronj0 commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author

The failing tests on alma10 is unrelated and visible on master: https://github.com/root-project/root/actions/runs/24824068443/job/72655489006

@aaronj0 aaronj0 merged commit c8626da into root-project:master Apr 23, 2026
70 of 77 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