Skip to content

Conversation

ferdymercury
Copy link
Collaborator

@ferdymercury ferdymercury commented Oct 13, 2025

Fixes #10556

This explores the solution proposed by @amadio (credit goes to him).

I tested that it works fine locally with reproducer #10556 (comment)

@ferdymercury ferdymercury requested a review from amadio October 13, 2025 17:01
@ferdymercury ferdymercury changed the title [cling] fallback solution when gcc is a soft link to ccache [cling] fallback compiler path when gcc is a soft link to ccache Oct 13, 2025
Copy link

github-actions bot commented Oct 13, 2025

Test Results

    22 files      22 suites   3d 19h 50m 27s ⏱️
 3 692 tests  3 692 ✅ 0 💤 0 ❌
79 273 runs  79 273 ✅ 0 💤 0 ❌

Results for commit b46ff63.

♻️ This comment has been updated with latest results.

Copy link
Member

@vgvassilev vgvassilev left a comment

Choose a reason for hiding this comment

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

LGTM!

@ferdymercury
Copy link
Collaborator Author

@guitargeek is the Roofit test failure in Windows "ok" ?

Copy link
Member

@amadio amadio left a comment

Choose a reason for hiding this comment

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

Thanks!

@guitargeek
Copy link
Contributor

@guitargeek is the Roofit test failure in Windows "ok" ?

Not really, it's the first time I see it. Thanks for letting me know! But it seem not related to this PR at least. I restarted the tests to check.

@ferdymercury
Copy link
Collaborator Author

I restarted the tests to check.

It seems it's still there, weird.

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.

ROOT cannot read compiler includes when compiled with a ccache compiler wrapper

4 participants