Skip to content

Fix stressGUI#22193

Open
bellenot wants to merge 4 commits into
root-project:masterfrom
bellenot:fix-stressgui
Open

Fix stressGUI#22193
bellenot wants to merge 4 commits into
root-project:masterfrom
bellenot:fix-stressgui

Conversation

@bellenot
Copy link
Copy Markdown
Member

@bellenot bellenot commented May 8, 2026

No description provided.

@bellenot bellenot self-assigned this May 8, 2026
@bellenot bellenot requested a review from dpiparo as a code owner May 8, 2026 15:13
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

Test Results

    22 files      22 suites   3d 11h 7m 54s ⏱️
 3 855 tests  3 855 ✅ 0 💤 0 ❌
76 132 runs  76 132 ✅ 0 💤 0 ❌

Results for commit bdbbfe7.

♻️ This comment has been updated with latest results.

@bellenot bellenot requested a review from couet as a code owner May 13, 2026 07:54
@bellenot bellenot closed this May 18, 2026
@bellenot bellenot reopened this May 18, 2026
@bellenot bellenot added the clean build Ask CI to do non-incremental build on PR label May 18, 2026
Comment thread test/stressGUI.cxx
bexec(dir, "tree500_cernbuild.C");
}
dir = gRootSys + "/tutorials/gui";
dir = gRootSys + "/tutorials/visualisation/gui";
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe here and in other places use TROOT::GetTutorialDir(). Like:

Suggested change
dir = gRootSys + "/tutorials/visualisation/gui";
dir = TROOT::GetTutorialDir() + "/visualisation/gui";

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good point! Let me change that...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@linev It would require gSystem->UnixPathName(TROOT::GetTutorialDir()) and then I got error C2110: '+': cannot add two pointers. SO I'll keep it the way it is now and I'll change this later

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Up to you.
I will try to enable stressGUI for CI - using xvfb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants