Skip to content

Fix dependency of functor cache and GaudiConfig2

Rosen Matev requested to merge rmatev-functor-cache into master

The change introduced with LHCb!2537 (merged) (+ the fixed typo in LHCb!2670 (merged)) would have never worked since the MergedConfDB2 target is only added after subdirectories (and the if(TARGET MergedConfDB2) in loki_functors_cache()) are processed. In addition, it was incorrectly passed to gaudirun.py.

Here we explicitly pass the MergedConfDB2 target to the dependencies of the functor cache. This should once and for all remove the race condition in the builds resulting in what's described in LHCb!2670 (merged).

The drawback is that we won't be able to build the functor cache as part of a lb-dev project. However, this is anyway something that wouldn't work today since the Rec functor cache definitions refer to CMake variable(s) that are only defined when building the full project.

Related to LHCb!2692 (merged)

Edited by Rosen Matev

Merge request reports