Tim Armstrong has posted comments on this change.

Change subject: IMPALA-4652: Add crcutil to toolchain
......................................................................


Patch Set 1:

IMO we should just add autotools as a toolchain package. We could install it 
and add it to the path as part of init.sh like cmake and binutils.

We don't really have a firm guarantee that other distros come with specific 
versions of autotools so that would be another step towards the toolchain 
builds being more reproducible

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf8b39082914b1b2932b4ce7efbd2cc4f5f69743
Gerrit-PatchSet: 1
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Henry Robinson <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: No

Reply via email to