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

Unified Diff: tools/update-doxygen.sh

Issue 14772003: Housekeeper should not fail when skia-autogen is reset (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Created 7 years, 8 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/update-doxygen.sh
===================================================================
--- tools/update-doxygen.sh (revision 8926)
+++ tools/update-doxygen.sh (working copy)
@@ -13,6 +13,16 @@
# export DOXYGEN_COMMIT=false
# bash update-doxygen.sh
+function check_out_docs {
+ svn checkout https://skia-autogen.googlecode.com/svn/docs # writeable
+ ret_code=$?
+ if [ $ret_code != 0 ]; then
+ # docs directory does not exist, skia-autogen must have been reset.
+ # Create a non svn docs directory instead.
+ mkdir docs
borenet 2013/05/01 12:10:39 Should you "svn add" docs here? I would think tha
rmistry 2013/05/01 12:15:22 Yes svn status will fail since its not yet a worki
+ fi
+}
+
# Prepare a temporary dir and check out Skia trunk and docs.
cd
DOXYGEN_TEMPDIR=${DOXYGEN_TEMPDIR:-/tmp/skia-doxygen}
@@ -28,12 +38,20 @@
fi
if [ -d "docs" ]; then
svn update --accept theirs-full docs
+ svn info docs
+ ret_code=$?
+ if [ $ret_code != 0 ]; then
+ # This is not a valid SVN checkout.
+ rm -rf docs
+ check_out_docs
+ fi
else
- svn checkout https://skia-autogen.googlecode.com/svn/docs # writeable
+ check_out_docs
+fi
+
if [ ! -f "docs/static_footer.txt" ]; then
cp trunk/tools/doxygen_footer.txt docs/static_footer.txt
fi
-fi
# Run Doxygen.
cd trunk
« 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