Hello Ashwani Raina, Yingchun Lai, Kudu Jenkins,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: [build-support] allow building with gcc/g++-7 on SLES
......................................................................

[build-support] allow building with gcc/g++-7 on SLES

SLES15 has GCC7 as the system compiler, so Kudu can be built with it
instead of installing GCC8 packages from the Development Tools Module.
If both GCC7 and GCC8 are present, prefer GCC8 over GCC7 to build Kudu.

This patch also updates the enable_devtoolset.sh script to properly
handle the absence of the `lsb_release` binary.

Change-Id: I4fd08cfb1cdc336b44b9568cb4dc019575f01efa
Reviewed-on: http://gerrit.cloudera.org:8080/19778
Tested-by: Kudu Jenkins
Reviewed-by: Ashwani Raina <ara...@cloudera.com>
Reviewed-by: Yingchun Lai <laiyingc...@apache.org>
(cherry picked from commit 7747a7c0149d6e8f894ad3216c54db9f27b3d372)
---
A build-support/ccache-gcc-7/c++
A build-support/ccache-gcc-7/cc
M build-support/enable_devtoolset.sh
3 files changed, 86 insertions(+), 23 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.17.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4fd08cfb1cdc336b44b9568cb4dc019575f01efa
Gerrit-Change-Number: 20056
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ara...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>

Reply via email to