[Nikita Borodikhin]
> --- /usr/bin/svn-bisect.orig    2010-04-20 13:57:31.000000000 +0700
> +++ /usr/bin/svn-bisect 2010-05-20 10:44:33.575998629 +0700
> @@ -55,7 +55,7 @@ do_reset ()
>      fi
>      url=$(cat $dir/url)
>      rev=$(cat $dir/start)
> -    svn switch -r$rev $url
> +    svn switch -r$rev $...@$rev
>      rm -fr $dir
>      echo "Now at r$rev in $url"
>  }
> @@ -128,7 +128,7 @@ do_good_bad ()
>      esac
>      target=$(head -n $(((n+1)/2)) $dir/revs | tail -n1)
>      echo "Switching to r$target ..."
> -    svn switch -r$target "$url"
> +    svn switch -r$target "$...@$target"
>      url2=$(svn_info URL)

Looks good - except, I'm pretty sure in both cases we want the same
peg, from $dir/start.  Otherwise you can't bisect a range that includes
creating or renaming the branch.

Here's what I'll commit as soon as I have a chance to test its basic
functionality (probably in 12 hours or so).  Thanks again,

Peter


--- bin/svn-bisect
+++ bin/svn-bisect
@@ -55,7 +55,7 @@
     fi
     url=$(cat $dir/url)
     rev=$(cat $dir/start)
-    svn switch -r$rev $url
+    svn switch -r$rev "$...@$rev"
     rm -fr $dir
     echo "Now at r$rev in $url"
 }
@@ -109,6 +109,7 @@
     good=$(head -n1 $dir/revs)
     bad=$(tail -n1 $dir/revs)
     url=$(cat $dir/url)
+    start=$(cat $dir/start)
     n=$(wc -l < $dir/revs)
     case $n in
         0)
@@ -128,7 +129,7 @@
     esac
     target=$(head -n $(((n+1)/2)) $dir/revs | tail -n1)
     echo "Switching to r$target ..."
-    svn switch -r$target "$url"
+    svn switch -r$target "$...@$start"
     url2=$(svn_info URL)
     if [ "$url" != "$url2" ]; then
         echo "r$target is in $url2"



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to