You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Michael Brown (Code Review)" <ge...@cloudera.org> on 2016/03/10 23:03:49 UTC

[Toolchain-CR] fix broken bzip2 executable symlinks and make them relative and relocatable

Michael Brown has uploaded a new change for review.

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

Change subject: fix broken bzip2 executable symlinks and make them relative and relocatable
......................................................................

fix broken bzip2 executable symlinks and make them relative and relocatable

The following bzip2 package artifacts are broken symlinks that only resolve on
the Cloudera machine that builds the toolchain:

  bzless
  bzfgrep
  bzegrep
  bzcmp

For example, if you're using the toolchain, you might find a symlink
similar to:

$ file bzip2-1.0.6-p1/bin/bzless
bzip2-1.0.6-p1/bin/bzless: broken symbolic link to /data/jenkins/bzip2-1.0.6-p1/bin/bzmore
$

bzip2 is being built per its directions, but the symlinks generated here
aren't relocatable, as demonstrated above.  Since these symlinks need to
point to targets in the same directory, one simple fix is to make the
symlinks relative, not absolute:

lrwxrwxrwx 1 mikeb mikeb 6 Mar  9 16:32 bzless -> bzmore

This is a pretty typical convention.

Testing:

Both `./build.sh bzip2 1.0.6-p2` and `./buildall.sh` succeeded. The
symlinks are relative:

$ ls -l build/bzip2-1.0.6-p2/bin/{bzless,bz?grep,bzcmp}
lrwxrwxrwx 1 mikeb mikeb 6 Mar 10 09:45 build/bzip2-1.0.6-p2/bin/bzcmp -> bzdiff
lrwxrwxrwx 1 mikeb mikeb 6 Mar 10 09:45 build/bzip2-1.0.6-p2/bin/bzegrep -> bzgrep
lrwxrwxrwx 1 mikeb mikeb 6 Mar 10 09:45 build/bzip2-1.0.6-p2/bin/bzfgrep -> bzgrep
lrwxrwxrwx 1 mikeb mikeb 6 Mar 10 09:45 build/bzip2-1.0.6-p2/bin/bzless -> bzmore
$

As a last test, I successfully built Impala using bzip-1.0.6-p2 (copied
into $IMPALA_TOOLCHAIN, not via s3).

Change-Id: Ic779d09805be2956491b7cfbb49c3d5ada300ec3
---
M buildall.sh
A source/bzip2/bzip2-1.0.6-patches/002-directoryless-executable-symlinks.diff
2 files changed, 22 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Toolchain refs/changes/25/2525/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2525
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic779d09805be2956491b7cfbb49c3d5ada300ec3
Gerrit-PatchSet: 1
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Michael Brown <mi...@cloudera.com>