Skip to content

Conversation

linev
Copy link
Member

@linev linev commented Aug 18, 2025

Like in cmsUnload, provide special runscript.C which loads commands and invoke them one by one.

Tests failing on most platforms with assert message from clang, to be investigated

@linev linev requested a review from pcanal August 18, 2025 12:40
@linev linev self-assigned this Aug 18, 2025
@linev linev requested review from bellenot and dpiparo as code owners August 18, 2025 12:40
@@ -1,5 +1,5 @@
.L lib/libjet.so
.L lib/libbtag.so
gSystem->Load("libjet_dictrflx");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am curious, why those changes?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First of all, .so is not working on Windows.
And second - use of .L libjet_dictrflx requires changes of default macro path.
So I trying to load library with gSystem->Load.
Test perfectly works on my node, but fails on all CI nodes.
So I need to investigate further

@pcanal
Copy link
Member

pcanal commented Aug 18, 2025

Like in cmsUnload, provide special runscript.C which loads commands and invoke them one by one.

If that PR is already open and/or merged, can we link it here?

Copy link

github-actions bot commented Aug 18, 2025

Test Results

    21 files      21 suites   3d 11h 7m 48s ⏱️
 3 571 tests  3 428 ✅  0 💤 143 ❌
73 238 runs  72 974 ✅ 17 💤 247 ❌

For more details on these failures, see this check.

Results for commit c061944.

♻️ This comment has been updated with latest results.

@linev
Copy link
Member Author

linev commented Aug 20, 2025

Like in cmsUnload, provide special runscript.C which loads commands and invoke them one by one.

If that PR is already open and/or merged, can we link it here?

Change of cmsUnload is here: #19672
But they do not depend from each other.
In both tests instead of redirecting of standard input one just read text file line by line and executing ProcessLine

linev added 3 commits August 22, 2025 14:25
Like in cmsUnload, provide special runscript.C which loads commands
and invoke them one by one.
Use of ProcessLine does not work on CI nodes
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