Skip to content

[cppinterop] Point cppinterop-version.tag at the ROOT-cppjit branch - #23603

Merged
aaronj0 merged 1 commit into
root-project:masterfrom
aaronj0:cppinterop-diff-root-cppjit
Oct 3, 2026
Merged

aaronj0 merged 1 commit into
root-project:masterfrom
aaronj0:cppinterop-diff-root-cppjit

Conversation

@aaronj0

@aaronj0 aaronj0 commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

The vendored tree carries ROOT patches on top of upstream, so it is mirrored verbatim on the ROOT-cppjit branch of compiler-research/CppInterOp. The diff workflow compares against that commit.

To be rebased on #21261, e xpected to fail until then

@aaronj0 aaronj0 self-assigned this Oct 3, 2026
@aaronj0
aaronj0 requested a review from vgvassilev as a code owner October 3, 2026 12:10
@aaronj0 aaronj0 added the in:CI label Oct 3, 2026
@aaronj0
aaronj0 requested a review from guitargeek October 3, 2026 12:16
@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown

Test Results

2 626 tests   2 626 ✅  1h 13m 51s ⏱️
    1 suites      0 💤
    1 files        0 ❌

Results for commit 046eb22.

The vendored tree carries ROOT patches on top of upstream, so it is
mirrored verbatim on the ROOT-cppjit branch of compiler-research/CppInterOp.
The diff workflow compares against that commit.
@aaronj0
aaronj0 force-pushed the cppinterop-diff-root-cppjit branch from 046eb22 to f34e140 Compare October 3, 2026 12:51

@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!

@aaronj0

aaronj0 commented Oct 3, 2026

Copy link
Copy Markdown
Contributor Author

merging, as this PR does not functionally change any code.

@aaronj0
aaronj0 merged commit b9613a8 into root-project:master Oct 3, 2026
12 of 36 checks passed
@aaronj0
aaronj0 deleted the cppinterop-diff-root-cppjit branch October 4, 2026 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants