Skip to content

[rootest] improve cmake of root/meta tests #19589

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 27 commits into
base: master
Choose a base branch
from

Conversation

linev
Copy link
Member

@linev linev commented Aug 8, 2025

  1. Convert reflex dictionary for meta/genreflex/ROOT-5768 into cmake, delete unused files there
  2. Remove many unused Makefile and .rootrc
  3. Use fixtures where possible

@linev linev self-assigned this Aug 8, 2025
@linev linev requested review from bellenot and dpiparo as code owners August 8, 2025 15:40
@linev linev marked this pull request as draft August 8, 2025 15:40
@linev linev force-pushed the roottest_meta branch 2 times, most recently from 2865872 to fce8afe Compare August 8, 2025 19:54
Copy link

github-actions bot commented Aug 8, 2025

Test Results

    21 files      21 suites   3d 11h 12m 18s ⏱️
 3 274 tests  3 259 ✅ 0 💤  15 ❌
67 028 runs  66 893 ✅ 0 💤 135 ❌

For more details on these failures, see this check.

Results for commit 05b483c.

♻️ This comment has been updated with latest results.

linev added 19 commits August 11, 2025 13:52
Use fixture and explicit tests creation
By this reactivate ROOT-5759 test
First generate all dicitonaries
Provide runscript.C macro which executes line by line
existing scripts from scripts/ sub-directory
Minimally adjust ref files
Introduce fixtures
Always provide output for genreflex to avoid creation of
temporary files in source directory
Remove old Makefile, .rootrc file and configure fixtures
It is generation of reflex dictionary for huge PyCool project
Check only if dictionary generated and compiled into static library
Remove lot of unused files
Configure missing stltest2.ref file
Remove unused teststl.ref
Provide separate windows ref files for foreign test
Add clear dependency via fixtures
@linev linev force-pushed the roottest_meta branch 2 times, most recently from 4aa69ff to b96181b Compare August 11, 2025 13:39
Like in cmsUnload, provide special runscript.C which loads commands
and invoke them one by one.
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.

1 participant