Skip to content

Conversation

linev
Copy link
Member

@linev linev commented Jun 24, 2025

Use cmake in several roottest/root/io tests:

  • io/newdelete
  • io/pointers
  • io/prefetching
  • io/recover
  • io/treeForeign

Resolve different compiler warnings for CI platform with -Ddev=On

@linev linev requested review from bellenot and dpiparo as code owners June 24, 2025 09:34
Copy link

github-actions bot commented Jun 24, 2025

Test Results

    20 files      20 suites   3d 4h 59m 19s ⏱️
 3 174 tests  3 174 ✅ 0 💤 0 ❌
61 883 runs  61 883 ✅ 0 💤 0 ❌

Results for commit 3ac698d.

♻️ This comment has been updated with latest results.

@linev linev force-pushed the roottest_io_scratch branch from c9aa75f to 0f02126 Compare June 24, 2025 12:35
@linev linev marked this pull request as draft June 24, 2025 12:36
@linev linev force-pushed the roottest_io_scratch branch 6 times, most recently from b7f4c5b to 0dad62a Compare June 30, 2025 14:05
@linev linev force-pushed the roottest_io_scratch branch from 0dad62a to 50ad695 Compare July 2, 2025 10:12
linev added 6 commits July 3, 2025 14:30
Remove several unused classes like BadClasses.h and TOffset.h/cxx
Build dictionary for all classes at once - no need for separate linkdefs
Make more unique name for library
Use fixtures
Mark as longtest
Like:
```
if (&rhs) {} // avoid warning about unused parameter
```
This makes warnings with new compilers
Avoid compilation of macro in source directory
@linev linev force-pushed the roottest_io_scratch branch from 50ad695 to 3ede0dc Compare July 3, 2025 12:40
@linev linev changed the title [roottest] testing next changes [roottest] use cmake for io tests [part4] Jul 3, 2025
@linev linev requested a review from pcanal July 3, 2025 12:40
@linev linev marked this pull request as ready for review July 3, 2025 12:40
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.

Thanks.

@linev linev closed this Jul 3, 2025
@linev linev reopened this Jul 3, 2025
@linev linev merged commit 31e52a4 into root-project:master Jul 4, 2025
51 of 70 checks passed
@linev linev deleted the roottest_io_scratch branch July 4, 2025 06:27
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