Repository navigation
Conversation
For each entry in DEPENDENCIES, ROOT_GENERATE_DICTIONARY passes rootcling
a `-m <dep>.pcm` flag and a dependency on the dependency's module file, so
that the prebuilt module is loaded instead of being built implicitly. This
was wired up by inspecting the target graph while the macro runs
(`if(NOT TARGET G__${dep})` / `get_target_property(... ROOT_PCM_FILENAME)`),
which only works if <dep> is configured before the dictionaries using it.
That breaks for core/multiproc: libMultiProc depends on Net (via TSocket),
but it lives under core/, configured long before net/. So when G__MultiProc
is generated, G__Net does not exist yet, the dependency is silently dropped,
and rootcling can build Net implicitly, failing with "Building module 'Net'
implicitly ... 'G__MultiProc.cxx' depends on 'Net'". The same shape affects
every dictionary depending on a module configured later (Tree -> Net, ...).
Decide this at generation time instead, when all targets exist: guard the
flag and the module-file dependency with $<TARGET_EXISTS:G__<dep>>. They are
emitted whenever <dep> provides a dictionary and expand to nothing otherwise
(e.g. a plain library such as TBB::tbb), regardless of configuration order.
Closes root-project#21673.
🤖 Problem identified by AI, then the human guided the AI to write the
solution that the human wanted to apply.
By pcanal They depend on it implicitly and not expressing leads to missing rebuilds as seen at: root-project#23046 when no ARG_DEPENDENCIES are specified at all Co-author: FerdyMercury
pcanal
changed the title
Test Results 23 files 23 suites 3d 15h 24m 38s ⏱️ For more details on these failures, see this check. Results for commit d955773. |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backport of #23207 and #22648 helping stabilize the CI (see for example failure https://github.com/root-project/root/actions/runs/42192142878/job/127819226593?pr=24024 and https://github.com/root-project/root/actions/runs/42192142878/job/128099437192?pr=24024)