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

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: s/uname/OS/ 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 20f1ffae53d900abfc47a6e8a95164035468e2af..f7200748f8dd6c86c3d3cc3e015b3617d819dba3 100755
--- a/tools/clang/scripts/update.sh
+++ b/tools/clang/scripts/update.sh
@@ -79,6 +79,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 [[ "${OS}" = "Darwin" ]]; then
+ XCODEBUILD_DIR="${THIS_DIR}/../../../xcodebuild"
+ MAKE_DIR="${THIS_DIR}/../../../out"
+ for CONFIG in Debug Release; do
+ if [[ -d "${MAKE_DIR}/${CONFIG}" ]]; then
Mark Mentovai 2011/09/15 02:53:46 Want to include /obj.target here too?
Nico 2011/09/15 03:00:08 Done.
+ echo "Clobbering ${CONFIG} PCH files for make build"
+ find "${MAKE_DIR}/${CONFIG}/obj.target" -name '*.gch' | xargs rm
Mark Mentovai 2011/09/15 02:53:46 find "${MAKE_DIR}/${CONFIG}/obj.target" -name '*.g
Nico 2011/09/15 03:00:08 Done. (DIdn't know about '+' up until now, thanks!
+ fi
+
+ if [[ -d "${XCODEBUILD_DIR}/${CONFIG}" ]]; then
Mark Mentovai 2011/09/15 02:53:46 Want to include /SharedPrecompiledHeaders here too
Nico 2011/09/15 03:00:08 Done.
+ echo "Clobbering ${CONFIG} PCH files for xcode build"
Mark Mentovai 2011/09/15 02:53:46 Nit: capital X in Xcode.
Nico 2011/09/15 03:00:08 Done.
+ rm -rf "${XCODEBUILD_DIR}/${CONFIG}"/SharedPrecompiledHeaders/*
Mark Mentovai 2011/09/15 02:53:46 Drop the /*. It’s OK to just remove all of SharedP
Nico 2011/09/15 03:00:08 Done.
+ 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