Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16798


Change subject: [thirdparty] fix preflight check on macOS 10.13
......................................................................

[thirdparty] fix preflight check on macOS 10.13

With e432e637e changelist, the pre-flight check on my macOS HighSierra
faled while trying to rebuild thirdparty components:

  Checking for C++17 compiler support ...
  *** Unable to compile a simple c++17 program. Please use g++ 7.0 or higher. 
On CentOS 7 or RHEL 7 you must use the devtoolset. See docs/installation.adoc 
for more info.
  Compilation failed: <stdin>:5:19: error: call to unavailable member function 
'value': introduced in macOS 10.14
          return oi.value();

This patch fixes the issue, switching from std::optional<T>::value()
to std::optional::operator*() in the pre-flight C++17 check.

For posterity, the environment was:

  $ sw_vers
  ProductName:  Mac OS X
  ProductVersion:       10.13.6
  BuildVersion: 17G14042

  $ clang --version
  Apple LLVM version 10.0.0 (clang-1000.11.45.5)
  Target: x86_64-apple-darwin17.7.0
  Thread model: posix
  
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Change-Id: I5c6f8b13f141d46d745011cd00f65d3440ab9016
---
M thirdparty/preflight.py
1 file changed, 3 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/98/16798/1
--
To view, visit http://gerrit.cloudera.org:8080/16798
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5c6f8b13f141d46d745011cd00f65d3440ab9016
Gerrit-Change-Number: 16798
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to