[core] no longer provide transitive unused includes

[ferdymercury]

No description provided.

[github-actions]

Test Results

    23 files      23 suites   3d 20h 13m 36s ⏱️
 3 878 tests  3 878 ✅ 0 💤 0 ❌
80 076 runs  80 076 ✅ 0 💤 0 ❌

Results for commit a2988e2.

[linev]

R_LESS_INCLUDES was introduced long time ago, but was never active in installed ROOT.
One should be careful with such changes - it can break compilation of user code without any warnings
And inexperienced users may fail to fix such simple compilation problem.

Maybe such change will be fine after ROOT 6.44 as part of ROOT7 changes.

[ferdymercury]

Thanks for the feedback!

And inexperienced users may fail to fix such simple compilation problem

From my latest experience, inexperienced users ask first LLMs before asking in the ROOT user forum, and they will get most probably the right answer.

Maybe such change will be fine after ROOT 6.44 as part of ROOT7 changes.

If that's the case, then I can mark this one as draft and suggest instead: #23366

[linev]

From my latest experience, inexperienced users ask first LLMs before asking in the ROOT user forum, and they will get most probably the right answer.

I face a lot of users who fail with such simple problem. And they just jump back to previous ROOT version or even stayed with ROOT5 until 2024 - 10 years after ROOT6 release.

[ferdymercury]

after ROOT 6.44

Could you create a label "ROOT6.46" or "ROOT7" so that I can mark this PR with that one? Thanks in advance!

[silverweed]

Could you create a label "ROOT6.46" or "ROOT7" so that I can mark this PR with that one? Thanks in advance!

@dpiparo

(I would be in favor of this change for ROOT7)

[hageboeck]

after ROOT 6.44

Could you create a label "ROOT6.46" or "ROOT7" so that I can mark this PR with that one? Thanks in advance!

I thought it better to make it a milestone; done now.