Skip to content

Commit

Permalink
[mcrouter] adjust tarball construction
Browse files Browse the repository at this point in the history
`tar cf "ARCHIVE_FILE" .` has a strange side effect that it includes a
reference to the current dir (which given the syntax, isn't unexpected I
guess), but in practice that mangles the paths in the archive to prefix
them all with "./" and confounds the permissions on the target directory
when you extract the archive.
  • Loading branch information
Paul Groudas committed May 24, 2021
1 parent 5fa72da commit a9db3dd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion mcrouter/scripts/slim-archive.sh
Original file line number Diff line number Diff line change
Expand Up @@ -29,4 +29,4 @@ LD_LIBRARY_PATH="${INSTALL_DIR}"/lib ldd "${INSTALL_DIR}"/bin/mcrouter "${INSTAL
| xargs -I '%' cp '%' "${ARCHIVE_INSTALL_DIR}"/lib
cp "${INSTALL_DIR}"/bin/mcrouter "${INSTALL_DIR}"/bin/mcpiper "${ARCHIVE_INSTALL_DIR}"/bin/

tar czfv "${ARCHIVE}" -C "${ARCHIVE_INSTALL_DIR}" .
tar czfv "${ARCHIVE}" -C "${ARCHIVE_INSTALL_DIR}" bin lib

0 comments on commit a9db3dd

Please sign in to comment.