Skip to content

[roottest] build libEvent with cmake [work in progress] #19173

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

[roottest] build libEvent with cmake [work in progress] #19173

wants to merge 11 commits into from

Conversation

linev
Copy link
Member

@linev linev commented Jun 25, 2025 •
edited
Loading

  1. Convert io/event to cmake
  2. Try to reuse in all dependent components
  3. While many Makefiles depends on libEvent target just because of sub-directories, one has to resolve all such tests first

Work in progress

@linev linev self-assigned this Jun 25, 2025
@linev linev requested review from bellenot and dpiparo as code owners June 25, 2025 11:01
@linev linev marked this pull request as draft June 25, 2025 11:01
@linev linev changed the title [rootest] build libEvent with cmake [work in progress] [roottest] build libEvent with cmake [work in progress] Jun 25, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 25, 2025 •
edited
Loading

Test Results

    20 files      20 suites   3d 16h 25m 9s ⏱️
 3 173 tests  3 166 ✅   0 💤  7 ❌
61 857 runs  61 482 ✅ 282 💤 93 ❌

For more details on these failures, see this check.

Results for commit 8b00a40.

♻️ This comment has been updated with latest results.

linev added 7 commits June 27, 2025 11:17
@linev
[roottest] use cmake with io/event tests
7a6f42a 
Keep dt_Makefile to let use Makefile-based tests in different places
@linev
[roottest] use libTestIoEvent in tree/cache test
9096ad2 
Instead build new library and manage own sources -
just reuse existing library. As it was before used in makefile
@linev
[roottest] reactivate root/treeformula/event with cmake
035010e 
Use central libTestIoEvent from `roottest/root/io/event`
Compile macros, generate several files and one reference file
Run drawing with several modes
@linev
[roottest] extend Event lib to be usable for root/treeformula
d4a45ff
@linev
[roottest] use cmake for treeformula/retobj tests
b30e1a4 
Reuse libTestIoEvent with small modifications
Simplify runretobjTest.C to avoid testing old CINT problems
@linev
[roottest] use cmake for treeformula/sync
e785b84
@linev
[roottest] let store output of command also when reference not provided
d7ad76c
linev added 4 commits June 27, 2025 16:08
@linev
[roottest] use cmake for treeformula/schemaEvolution
263d039
@linev
[roottest] support cc and cpp extension for MACRO
0cfe6d8
@linev
[roottest] use cmake for tree/friend tests
ce0fa5d 
Re-enable several tests like all testFriend.C macros
Delete unused files
@linev
[roottest] re-enable tree/fastcloningeventtree
8b00a40 
Very similar to treeformula/event testing,
therefore reuse code from there
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bellenot bellenot Awaiting requested review from bellenot bellenot is a code owner

@dpiparo dpiparo Awaiting requested review from dpiparo dpiparo is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@linev linev

Labels
affects:master in:I/O in:Testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@linev