Adar Dembo has submitted this change and it was merged. Change subject: preflight: require libtoolize on Linux, not libtool ......................................................................
preflight: require libtoolize on Linux, not libtool Modern Ubuntu-based distros have split libtool into an architecture independent package (libtool) and a dependent package (libtool-bin). Only the former is needed to build thirdparty, and it contains the binary 'libtoolize', not 'libtool', so let's check for that. Older Linux distros include both, so this change shouldn't affect them. Change-Id: I590c9fbdc920da100aa3e4a9cd8cca8847569968 Reviewed-on: http://gerrit.cloudera.org:8080/2646 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M thirdparty/preflight.py 1 file changed, 9 insertions(+), 1 deletion(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I590c9fbdc920da100aa3e4a9cd8cca8847569968 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
