Skip to content

[cmake] Don't serialize roottest builds with MSVC#21670

Open
bellenot wants to merge 3 commits intoroot-project:masterfrom
bellenot:fix-roottest-vs-parallel-build
Open

[cmake] Don't serialize roottest builds with MSVC#21670
bellenot wants to merge 3 commits intoroot-project:masterfrom
bellenot:fix-roottest-vs-parallel-build

Conversation

@bellenot
Copy link
Member

After commit #6436fbdc09a9 the testing time increased by factor 2~3
This change partially revert it (only for Visual Studio)

After commit #6436fbdc09a9 the testing time increased by factor 2~3
This change partially revert it (only for Visual Studio)
@bellenot bellenot requested review from dpiparo and pcanal March 24, 2026 10:01
@bellenot bellenot self-assigned this Mar 24, 2026
@github-actions
Copy link

Test Results

    22 files      22 suites   3d 4h 20m 48s ⏱️
 3 835 tests  3 833 ✅ 1 💤 1 ❌
75 686 runs  75 676 ✅ 9 💤 1 ❌

For more details on these failures, see this check.

Results for commit 8a39db0.

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.

LGTM. Merci.

@dpiparo
Copy link
Member

dpiparo commented Mar 25, 2026

Thanks for the changes.

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.

3 participants