Skip to content

[roottest] use cmake for root/treedraw test#19534

Merged
linev merged 3 commits intoroot-project:masterfrom
linev:roottest_treedraw
Aug 5, 2025
Merged

[roottest] use cmake for root/treedraw test#19534
linev merged 3 commits intoroot-project:masterfrom
linev:roottest_treedraw

Conversation

@linev
Copy link
Member

@linev linev commented Aug 5, 2025

Use hsimple.C from tutorials directory
Adjust hsimple.ps.ref file to used hsimple
Remove rootlogon.C script and compile utils.cc macro directly
Adjust utils.cc to be able use it on Windows

Use hsimple.C from tutorials directory
Adjust hsimple.ps.ref file to used hsimple
Remove rootlogon.C script and compile utils.cc macro directly
Disable on Windows while utils.cc does not compiled there
@linev linev self-assigned this Aug 5, 2025
@linev linev requested a review from bellenot as a code owner August 5, 2025 08:54
@linev linev requested a review from dpiparo as a code owner August 5, 2025 08:54
@linev linev removed the request for review from dpiparo August 5, 2025 08:54
linev added 2 commits August 5, 2025 11:05
Allows to use this script on Windows
Extra redirection of debug output is not really necessary
Use c++ includes
Now utils.cc can be compiled on Windows as well
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

Test Results

    21 files      21 suites   3d 11h 53m 19s ⏱️
 3 226 tests  3 225 ✅ 0 💤 1 ❌
66 019 runs  66 018 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 581d741.

Copy link
Member

@bellenot bellenot left a comment

Choose a reason for hiding this comment

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

LGTM but this test is failing: gtest-tree-dataframe-datasource-root

@linev
Copy link
Member Author

linev commented Aug 5, 2025

LGTM but this test is failing: gtest-tree-dataframe-datasource-root

This failure is not related to PR

Copy link
Member

@bellenot bellenot left a comment

Choose a reason for hiding this comment

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

LGTM

@linev linev merged commit 59d16bf into root-project:master Aug 5, 2025
23 of 26 checks passed
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

Comments