Joe McDonnell has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/19846


Change subject: KUDU-3474: Add zlib as dependency of libunwind for ARM
......................................................................

KUDU-3474: Add zlib as dependency of libunwind for ARM

On ARM, libunwind includes support for compressed
.debug_info sections if zlib is available during
compilation. This means that libunwind can require zlib
at link time, but only for ARM architecture.

This modifies the build to add a zlib dependency for
libunwind if the architecture is ARM. It also reorders
the thirdparty build so that zlib is built before
libunwind. This is not strictly necessary, but it means
that libunwind would consistently have support for
compressed .debug_info sections on ARM. This does not
impact x86_64.

Testing:
 - Ran an ARM build on Ubuntu 20.04

Change-Id: Iee87b908a5771c2bd7c1a653504a1449d0792280
---
M CMakeLists.txt
M thirdparty/build-thirdparty.sh
2 files changed, 12 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/19846/2
--
To view, visit http://gerrit.cloudera.org:8080/19846
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iee87b908a5771c2bd7c1a653504a1449d0792280
Gerrit-Change-Number: 19846
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>

Reply via email to