Skip to content

[roottest] root/tree, part 6#19527

Merged
linev merged 7 commits intoroot-project:masterfrom
linev:roottest_tree_6
Aug 20, 2025
Merged

[roottest] root/tree, part 6#19527
linev merged 7 commits intoroot-project:masterfrom
linev:roottest_tree_6

Conversation

@linev
Copy link
Member

@linev linev commented Aug 4, 2025

This Pull request:

Changes or fixes:

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

This PR fixes #

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

github-actions bot commented Aug 4, 2025

Test Results

    20 files      20 suites   3d 6h 23m 55s ⏱️
 3 304 tests  3 304 ✅ 0 💤 0 ❌
64 390 runs  64 390 ✅ 0 💤 0 ❌

Results for commit b0c7758.

♻️ This comment has been updated with latest results.

linev added 6 commits August 5, 2025 15:48
Remove printout with pointer to allow tests on Windows,
otherwise `grep -v 0x` has to be involved which excludes Windows usage
Avoid duplication of dictionary when compile these macros on Windows
Build dictionary in convertIssues/ subdir and correspondent lib before running test
Remove exe bit from many sources
Remove rootlogon.C script - used only with runiobug.C, where necessary script load
with ROOTEXE_OPTS
On Mac platform may produce compiler warnings
Typical is compilation problem with dictionaries
@linev linev force-pushed the roottest_tree_6 branch from 795d470 to b0c7758 Compare August 5, 2025 14:12
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.

Co-authored-by: Philippe Canal <pcanal@fnal.gov>
@linev linev merged commit 65e8e28 into root-project:master Aug 20, 2025
22 of 25 checks passed
@linev linev deleted the roottest_tree_6 branch August 20, 2025 11:44
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