You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2019/09/14 00:06:21 UTC

[native-toolchain-CR] IMPALA-3926: fix RPATH for libstdc++.so and libgcc.so

Hello Tianyi Wang, Hector Acosta, Philip Zeyliger, Joe McDonnell, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/6521

to look at the new patch set (#7).

Change subject: IMPALA-3926: fix RPATH for libstdc++.so and libgcc.so
......................................................................

IMPALA-3926: fix RPATH for libstdc++.so and libgcc.so

C++ packages depend on these shared objects. Our toolchain packages
should always be run using the toolchain version of the shared
objects that it ships. Previously the toolchain artifacts were
often (but not always) linked against system version of libgcc.so
and libstdc++.so, which can cause compatibility problems. This
is a major problem on recent Linux distros like Ubuntu 16.04
that default to the new C++11 ABI because the system libstdc++.so
is not compatible with gcc-4.9.2-generated binaries. It also
means behaviour of toolchain artifacts may be less consistent
across systems.

This patch does two things to ensure that toolchain artifacts link
to the correct libraries:
1. fixes the RPATHs added to executables and shared objects so that
  they always point to the ../lib/ and ../lib64/ directories.
2. adds symlinks from the lib/ directory to libstd++.so and libgcc.so
  where required by executables and shared objects.

Change-Id: Ie3f8481a8dfe35273a763586e9d2da0d4008ac67
---
M functions.sh
M init-compiler.sh
M source/flatbuffers/build.sh
M source/kudu/build.sh
4 files changed, 121 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/native-toolchain refs/changes/21/6521/7
-- 
To view, visit http://gerrit.cloudera.org:8080/6521
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie3f8481a8dfe35273a763586e9d2da0d4008ac67
Gerrit-Change-Number: 6521
Gerrit-PatchSet: 7
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>
Gerrit-Reviewer: Hector Acosta <he...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tianyi Wang <tw...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>