Hello Kudu Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/18874

to look at the new patch set (#2).

Change subject: [cmake] cleanup on clang version mapping on macOS
......................................................................

[cmake] cleanup on clang version mapping on macOS

This patch cleans up the code that determines compiler type and version
on macOS for Xcode-based LLVM/CLANG.  As it turns out, there is a well
documented mapping for clang versions: see [1] and [2].

I also updated unified regex patterns used to find versions of the
compiler in its version string.

References:
  [1] https://trac.macports.org/wiki/XcodeVersionInfo
  [2] https://en.wikipedia.org/wiki/Xcode#Version_history

Change-Id: I656dd4b20c7c852746861861f0741763b02609d5
---
M cmake_modules/CompilerInfo.cmake
1 file changed, 83 insertions(+), 36 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/74/18874/2
--
To view, visit http://gerrit.cloudera.org:8080/18874
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I656dd4b20c7c852746861861f0741763b02609d5
Gerrit-Change-Number: 18874
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to