FreeBSD etc

[JamesMolson]

maybe extract the version from freebsd-version or from uname -r ?

[JamesMolson]

Or do the better thing, https://cmake.org/Wiki/CMake_Useful_Variables

CMAKE_SYSTEM 
    the complete system name, e.g. "Linux-2.4.22", "FreeBSD-5.4-RELEASE" or "Windows 5.1" 
CMAKE_SYSTEM_NAME 
    the short system name, e.g. "Linux", "FreeBSD" or "Windows" 

[JamesMolson]

elseif (${PLATFORM} MATCHES "FreeBSD")
	set ( PLATFORM ${CMAKE_SYSTEM})
endif()

is sufficient

[Sam-Tygier]

Could you confirm that commit ebf85c3 works on freebsd

For linux I think it is good to still have distro checking code, rather than just using CMAKE_SYSTEM

[JamesMolson]

http://abp-cdash.web.cern.ch/abp-cdash/buildSummary.php?buildid=32177

It works :)

[JamesMolson]

Well, some tests do fail, but the script runs :)

[JamesMolson]

Though:

-- The C compiler identification is Clang 3.8.0
-- The CXX compiler identification is Clang 3.8.0

And the script has reported it as gcc 4.9.4 which wasn't used.

[Sam-Tygier]

The bit in the reported string comes from the output of running ${CMAKE_CXX_COMPILER} --version
Looks to me like cmake is ignoring that and using the default compiler anyway.

So you could put set(CMAKE_CXX_COMPILER "clang") and get the version sting right. But it would keep using clang, no matter what the variable is set too.

[Sam-Tygier]

Looks like it just needed to be passed across 1a71878

[Sam-Tygier]

and of course it should be
set(CMAKE_CXX_COMPILER "clang++")

[JamesMolson]

Build name: FreeBSD-11.0-RELEASE-p2 amd64 clang++ 3.8.0 (Release)

That is better.

Also the cdash box is out of disk space now :p