impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tim Armstrong (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5739: Correctly handle sles12 SP2
Date Fri, 28 Jul 2017 16:51:02 GMT
Tim Armstrong has submitted this change and it was merged.

Change subject: IMPALA-5739: Correctly handle sles12 SP2
......................................................................


IMPALA-5739: Correctly handle sles12 SP2

This takes care of the difference in outputs for SLES 12 SP1 and SP2.
For reference here's the outputs in sles12sp1 and sp2:

sles12sp1 # lsb_release -irs
SUSE LINUX 12.1
sles12sp2 # lsb_release -irs
SUSE 12.2

Testing:
Did a full build on SLES12 SP2. Before this patch, a build resulted in:
'Pre-built toolchain archives not available for your platform.'

After this patch:
Toolchain bootstrap complete.

..Followed by a full build.

Change-Id: I005e05b8b66de78e6d53a35a894eb34d89843a62
Reviewed-on: http://gerrit.cloudera.org:8080/7535
Tested-by: Tim Armstrong <tarmstrong@cloudera.com>
Reviewed-by: Tim Armstrong <tarmstrong@cloudera.com>
---
M bin/bootstrap_toolchain.py
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Tim Armstrong: Looks good to me, approved; Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/7535
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I005e05b8b66de78e6d53a35a894eb34d89843a62
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: hector.acosta@cloudera.com <hector.acosta@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstrong@cloudera.com>

Mime
View raw message