Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2734)

Unified Diff: build_tools/common.sh

Issue 687823002: Don't package OSX's magic ._foo files (Closed) Base URL: https://chromium.googlesource.com/external/naclports.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build_tools/common.sh
diff --git a/build_tools/common.sh b/build_tools/common.sh
index 4527f7083e3828fadbffb111767f0f5236e96ce9..f37a66a9fe80b408d6f09a4b4767f060d074b1d9 100644
--- a/build_tools/common.sh
+++ b/build_tools/common.sh
@@ -1513,7 +1513,14 @@ PackageStep() {
echo "BUILD_TOOLCHAIN=${TOOLCHAIN}" >> "${INSTALL_DIR}/pkg_info"
echo "BUILD_SDK_VERSION=${NACL_SDK_VERSION}" >> "${INSTALL_DIR}/pkg_info"
echo "BUILD_NACLPORTS_REVISION=${FULL_REVISION}" >> "${INSTALL_DIR}/pkg_info"
- LogExecute tar cjf "${PACKAGE_FILE}" -C "${INSTALL_DIR}" .
+ if [ "${OS_NAME}" = "Darwin" ]; then
+ # OSX likes to create files starting with ._. We don't want to package
+ # these.
+ local args="--exclude ._*"
+ else
+ local args=""
+ fi
+ LogExecute tar cjf "${PACKAGE_FILE}" -C "${INSTALL_DIR}" ${args} .
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698