Change the way the checkpoint build system finds detector about magistrate HOT 3 CLOSED

When I #include <serialization_library_headers.h> in anything, I get a bunch of errors about stuff not existing in the std namespace. I wonder if this is related to compiler feature detection failing in DARMA-tasking/detector#1. I'm using Clang 3.9.0.

from magistrate.

Unfortunately, I have pretty much no familiarity with Cmake development, and would probably take longer to get up to speed than Jonathan will be away.

If we need something that will just work for now, perhaps checkpoint itself could embed the detector code directly, rather than as a sibling TPL?

As for the compiler version and its feature detection from Cmake, I've only been working with much newer GCC on the Linux clusters. I think it maybe avoids some messiness of Cmake getting flags wrong by assuming a newer language standard as the baseline default.

from magistrate.

When I bumped up the cmake_minimum_required on the shared build system and reinstated the compiler feature detection used by detector, these errors went away.

from magistrate.