Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/20056 )

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 <[email protected]>
Reviewed-by: Yingchun Lai <[email protected]>
(cherry picked from commit 7747a7c0149d6e8f894ad3216c54db9f27b3d372)
Reviewed-on: http://gerrit.cloudera.org:8080/20056
Reviewed-by: Alexey Serbin <[email protected]>
---
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(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
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: merged
Gerrit-Change-Id: I4fd08cfb1cdc336b44b9568cb4dc019575f01efa
Gerrit-Change-Number: 20056
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <[email protected]>

Reply via email to