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

Unified Diff: tools/clang/scripts/update.sh

Issue 7890052: mac: Clobber pch files after clang updates. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: +x Created 9 years, 3 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: tools/clang/scripts/update.sh
diff --git a/tools/clang/scripts/update.sh b/tools/clang/scripts/update.sh
index f3a261b4e4a955899875baa0b07796e26e968cd7..e4bf68b22af9b0d2fd51d5cfa1c926587ffdb5cc 100755
--- a/tools/clang/scripts/update.sh
+++ b/tools/clang/scripts/update.sh
@@ -60,6 +60,24 @@ fi
# To always force a new build if someone interrupts their build half way.
rm -f "${STAMP_FILE}"
+# Clobber pch files, since they only work with the compiler version that
+# created them.
+if [[ "$(uname -s)" = "Darwin" ]]; then
Nico 2011/09/14 21:31:34 I'll switch this to ${OS} once that other CL is in
+ XCODEBUILD_DIR="${THIS_DIR}/../../../xcodebuild"
+ MAKE_DIR="${THIS_DIR}/../../../out"
+ for CONFIG in Debug Release; do
+ if [[ -d "${MAKE_DIR}/${CONFIG}" ]]; then
+ echo "Clobbering ${CONFIG} PCH files for make build"
+ find "${MAKE_DIR}/${CONFIG}/obj.target" -name '*.gch' | xargs rm
+ fi
+
+ if [[ -d "${XCODEBUILD_DIR}/${CONFIG}" ]]; then
+ echo "Clobbering ${CONFIG} PCH files for xcode build"
+ rm -rf "${XCODEBUILD_DIR}/${CONFIG}"/SharedPrecompiledHeaders/*
+ fi
+ done
+fi
+
if [ -z "$force_local_build" ]; then
# Check if there's a prebuilt binary and if so just fetch that. That's faster,
# and goma relies on having matching binary hashes on client and server too.
« 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