diff options
| author | Brad King <brad.king@kitware.com> | 2017-02-27 15:30:20 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2017-02-27 15:30:45 (GMT) |
| commit | c1aaf8a61defe3e01e8526b99d8919b9618d1ba9 (patch) | |
| tree | d419eee5db59fcdb29b8b337fe00f50e6c35c1b8 | |
| parent | 717e1f3056b45e79dcb433b1098e8b6fd813d07b (diff) | |
| download | CMake-c1aaf8a61defe3e01e8526b99d8919b9618d1ba9.zip CMake-c1aaf8a61defe3e01e8526b99d8919b9618d1ba9.tar.gz CMake-c1aaf8a61defe3e01e8526b99d8919b9618d1ba9.tar.bz2 | |
Fix CMAKE_HOST_SYSTEM_NAME on SunOS
In commit 0bbd993f (Make CMAKE_HOST_SYSTEM_NAME available in scripting
context, 2016-12-26) we added a call to `uname` that checks for a zero
return value. However, on Solaris the `uname(2)` manual [1] says that
on success a non-negative value is returned. Fix our return code check
so that we detect the `SunOS` name correctly.
[1] https://docs.oracle.com/cd/E53394_01/html/E54765/uname-2.html
| -rw-r--r-- | Source/cmStateSnapshot.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Source/cmStateSnapshot.cxx b/Source/cmStateSnapshot.cxx index 80e494b..d2c9d73 100644 --- a/Source/cmStateSnapshot.cxx +++ b/Source/cmStateSnapshot.cxx @@ -308,7 +308,7 @@ void cmStateSnapshot::SetDefaultDefinitions() this->SetDefinition("CMAKE_HOST_UNIX", "1"); struct utsname uts_name; - if (uname(&uts_name) == 0) { + if (uname(&uts_name) >= 0) { this->SetDefinition("CMAKE_HOST_SYSTEM_NAME", uts_name.sysname); } #endif |