Skip to content

Conversation

@linev
Copy link
Member

@linev linev commented Aug 21, 2025

Compile macros in advance and use them from srun.C script
Avoid usage of rootlogon.C - which automatically involved for all scripts
Adjust stlIoTest.ref with actual warnings
Remove execute flag for many files
Mark as failing readNoLib.C - was fully disabled in Makefile

@github-actions
Copy link

github-actions bot commented Aug 21, 2025

Test Results

    22 files      22 suites   3d 21h 0m 42s ⏱️
 3 735 tests  3 734 ✅ 0 💤 1 ❌
80 219 runs  80 218 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit cd8023a.

♻️ This comment has been updated with latest results.

@linev linev changed the title [roottest] use cmake for io/newstl tests [roottest] use cmake for io/newstl tests, part 4 Aug 22, 2025
Compile macros in advance and use them from srun.C script
Avoid usage of rootlogon.C - which automatically involved for all scripts
Adjust stlIoTest.ref with actual warnings
Remove execute flag for many files
Mark as failing readNoLib.C - was fully disabled in Makefile
Windows required correct linking, therefore all depend libs must be linked
…tl test

For some reason warning "ROOT does not support variable size array of stl containers" appears for all kind of containers
Seems to be, required on Windows
@linev linev closed this Sep 15, 2025
@linev linev reopened this Sep 15, 2025
@linev linev requested a review from pcanal October 16, 2025 11:01
@pcanal pcanal closed this Oct 28, 2025
@pcanal pcanal reopened this Oct 28, 2025
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 for your patience.

@linev linev merged commit 6ae1c78 into root-project:master Nov 7, 2025
62 of 76 checks passed
@linev linev deleted the roottest_io_4 branch November 7, 2025 06:58
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