Skip to content

Fix how the Range-v3 include directory dependency is added

Olli Lupton requested to merge olupton_trackv2_rangev3 into master

Thanks to @clemenci for the fix.

Fix how the Range-v3 include directory dependency is added. This means it is correctly propagated to ROOT_INCLUDE_PATH in the runtime environment.

The original error was:

ParticleFunctor...   INFO Cache miss for functor: ::Functors::AcceptAll{}
ParticleFunctor...   INFO Using the interpreter for functor: ::Functors::AcceptAll{}
In file included from input_line_10:1:
In file included from /home/olupton/new-stack/Rec/Phys/FunctorCore/Functors/ClingHacks.h:12:
/home/olupton/new-stack/LHCb/InstallArea/x86_64+avx2+fma-centos7-gcc8-opt+g/include/Event/Track_v2.h:24:10: fatal error: 'range/v3/range_traits.hpp' file not found
#include "range/v3/range_traits.hpp"
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~
input_line_11:1:27: error: expected namespace name
using namespace Functors::ClingHacks;
                ~~~~~~~~~~^

because ROOT_INCLUDE_PATH did not include the Range-v3 headers, despite these being used by v2::Track.

@cattanem please un-WIP and merge as is convenient

cc: @apearce

Edited by Marco Cattaneo

Merge request reports