Joe McDonnell has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/19756 )

Change subject: IMPALA-12070: Use autotools/bison from the OS rather than 
building it
......................................................................

IMPALA-12070: Use autotools/bison from the OS rather than building it

This stops building autoconf, automake, bison, and libtool.
These utilities are already installed in the docker images,
so this removes any code that used the toolchain version
of these. This is useful from a license perspective, because
several of these are GPL licensed. Removing them avoids any
complications due to licensing, and it brings the native-toolchain
code closer to compatibility with Apache Impala.

This also removes unused build code for cyrus-sasl, krb5,
and openssl. This code remains in the history and can be
revived at any time.

Testing:
 - Ran a full toolchain build

Change-Id: I2f0603671fa443b26b3a339874f4154e853e2a06
Reviewed-on: http://gerrit.cloudera.org:8080/19756
Reviewed-by: Michael Smith <[email protected]>
Reviewed-by: Wenzhe Zhou <[email protected]>
Tested-by: Joe McDonnell <[email protected]>
---
M build-kudu-only.sh
M buildall.sh
M functions.sh
M init-compiler.sh
M init.sh
D source/autoconf/build.sh
D 
source/automake/automake-1.14.1-patches/0001-make-help2man-work-with-perl.patch
D source/automake/build.sh
D source/bison/bison-3.0.4-patches/110-glibc-change-work-around.patch
D source/bison/build.sh
M source/crcutil/build.sh
D source/cyrus-sasl/build.sh
D source/krb5/build.sh
D source/libtool/build.sh
D source/openssl/build.sh
M source/thrift/build.sh
16 files changed, 3 insertions(+), 392 deletions(-)

Approvals:
  Michael Smith: Looks good to me, but someone else must approve
  Wenzhe Zhou: Looks good to me, approved
  Joe McDonnell: Verified

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

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I2f0603671fa443b26b3a339874f4154e853e2a06
Gerrit-Change-Number: 19756
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>

Reply via email to