Skip to content

Fix for case USE_VECGEOM_CONTAINERS=OFF

John Apostolakis requested to merge fix-stdlib-containers into master

The current master fails to 'compile' in the case that USE_VECGEOM_CONTAINERS=OFF and Vc is the chosen backend.

It appears that the Materials library now uses backend functionality (an expected extension) which now involves Vc in the case of standard containers (but not for VecGeom container - a difference which we should ponder.)

The proposed fix simply aligns the libraries used to link the libMaterials.so in the case of StdLib containers with the libraries used when VecGeom containers are chosen. A more lightweight choice may be possible - or one which does not involve VecGeom, if we were to continue to support non-VecGeom builds.

I suggest, though, that a change/fix is needed NOW for the important case of StdLibs containers.

I can confirm that this fix works on Mac OS 10.12.5 with Xcode 8, and testing should reveal whether it is portable to all the other platforms.

Merge request reports