svn commit: r1092659 - /subversion/trunk/contrib/server-side/fsfsfixer/fix-repo

[ju...@apache.org]

Author: julianfoad
Date: Fri Apr 15 10:51:37 2011
New Revision: 1092659

URL: http://svn.apache.org/viewvc?rev=1092659&view=rev
Log:
* contrib/server-side/fsfsfixer/fix-repo
  For portability and scalability, iterate over revisions using Bash
  arithmetic rather than by calling "seq". Large revision numbers were being
  formatted like "2.123456e+06" on some platforms, presumably by "seq",
  which caused fix-rev.py to fail. This change should also avoid running
  into a command-line length limitation when iterating over a very large
  range of revision numbers.

Modified:
    subversion/trunk/contrib/server-side/fsfsfixer/fix-repo

Modified: subversion/trunk/contrib/server-side/fsfsfixer/fix-repo
URL: http://svn.apache.org/viewvc/subversion/trunk/contrib/server-side/fsfsfixer/fix-repo?rev=1092659&r1=1092658&r2=1092659&view=diff
==============================================================================
--- subversion/trunk/contrib/server-side/fsfsfixer/fix-repo (original)
+++ subversion/trunk/contrib/server-side/fsfsfixer/fix-repo Fri Apr 15 10:51:37 2011
@@ -11,11 +11,16 @@ if [ ! -d "$REPO_DIR" ] || [ "$START_REV
   exit 1
 fi
 
-YOUNGEST="$(svnlook youngest $REPO_DIR)"
+YOUNGEST="$(svnlook youngest "$REPO_DIR")"
+
+if [ "$YOUNGEST" = "" ]; then
+  echo "$0: error running 'svnlook youngest $REPO_DIR'"
+  exit 1
+fi
 
 echo "Verifying revisions $START_REV through $YOUNGEST."
 
-for REV in $(seq $START_REV $YOUNGEST); do
-  echo === r$REV
-  ./fixer/fix-rev.py "$REPO_DIR" $REV
+for ((REV=$START_REV; REV <= $YOUNGEST; REV++)); do
+  echo "=== r$REV"
+  ./fixer/fix-rev.py "$REPO_DIR" "$REV"
 done