We've been meaning to remove support for the legacy testing configuration
for a long time. This patch switches the default from the legacy config
to the appropriate new-style configuration based on a few hints.
We've been running with the new-style configuration for more than a year
in our CI, however it's possible that this will uncover issues with some
users that run the tests on platforms that we don't support yet with the
new-style configs. Unfortunately, there is no way to know about it other
than to land this patch and see whether anything breaks.
This is all a matter of taste, but I'd find elseif(MINGW) elseif(WIN32) #clang-cl elseif(... more straightforward, instead of the WIN32 AND NOT MINGW followed by the complement of the negation.
Alternatively, depending on which one to single out, one could also use elseif (MSVC) # clang-cl elseif (MINGW), as MSVC is defined in clang-cl configurations.