Skip to content

[roottest] use cmake for root/treeproxy tests#19565

Merged
linev merged 8 commits intoroot-project:masterfrom
linev:roottest_treeproxy
Aug 27, 2025
Merged

[roottest] use cmake for root/treeproxy tests#19565
linev merged 8 commits intoroot-project:masterfrom
linev:roottest_treeproxy

Conversation

@linev
Copy link
Member

@linev linev commented Aug 7, 2025

In many cases one need to copy scripts into build directory and
configure macro path with:

                  ROOTEXE_OPTS -e "(void)gROOT->SetMacroPath(\"${CMAKE_CURRENT_BINARY_DIR}\")"

Otherwise TTreeProxyGenerator not able to find scripts.

Also crucial to have some compiled macros in current directory to get correct declaration file set for created TClass instance - like in case of TTrack class. When not set - created selected will miss Track.h include and will not work.

@linev linev requested a review from pcanal August 7, 2025 09:24
@linev linev self-assigned this Aug 7, 2025
@linev linev requested review from bellenot and dpiparo as code owners August 7, 2025 09:24
@github-actions
Copy link

github-actions bot commented Aug 7, 2025

Test Results

    21 files      21 suites   3d 14h 10m 50s ⏱️
 3 644 tests  3 492 ✅  0 💤 152 ❌
74 769 runs  74 599 ✅ 18 💤 152 ❌

For more details on these failures, see this check.

Results for commit 04ba182.

♻️ This comment has been updated with latest results.

@linev linev closed this Aug 20, 2025
@linev linev reopened this Aug 20, 2025
@linev linev force-pushed the roottest_treeproxy branch from 9eb674e to 04ba182 Compare August 22, 2025 11:00
@pcanal pcanal closed this Aug 25, 2025
@pcanal pcanal reopened this Aug 25, 2025
@linev
Copy link
Member Author

linev commented Aug 26, 2025

@pcanal All failures unrelated to PR, they appears in all other PRs

Copy link
Member

@pcanal pcanal left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@linev linev merged commit 0a9fcf6 into root-project:master Aug 27, 2025
44 of 51 checks passed
@linev linev deleted the roottest_treeproxy branch August 27, 2025 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants