Merge git://git.bogomips.org/git-svn

gitster · gitster · commit 24fa13cbb7c5 · 2008-09-08T14:15:17.000-07:00
* git://git.bogomips.org/git-svn:
  git-svn: fix handling of even funkier branch names
diff --git a/git-svn.perl b/git-svn.perl
@@ -4025,6 +4025,7 @@ sub gs_do_switch {
 		}
 	}
 	$ra ||= $self;
+	$url_b = escape_url($url_b);
 	my $reporter = $ra->do_switch($rev_b, '', 1, $url_b, $editor, $pool);
 	my @lock = $SVN::Core::VERSION ge '1.2.0' ? (undef) : ();
 	$reporter->set_path('', $rev_a, 0, @lock, $pool);
diff --git a/t/t9118-git-svn-funky-branch-names.sh b/t/t9118-git-svn-funky-branch-names.sh
@@ -6,6 +6,10 @@
 test_description='git-svn funky branch names'
 . ./lib-git-svn.sh
 
+# Abo-Uebernahme (Bug #994)
+scary_uri='Abo-Uebernahme%20%28Bug%20%23994%29'
+scary_ref='Abo-Uebernahme%20(Bug%20#994)'
+
 test_expect_success 'setup svnrepo' '
 	mkdir project project/trunk project/branches project/tags &&
 	echo foo > project/trunk/foo &&
@@ -15,6 +19,8 @@ test_expect_success 'setup svnrepo' '
 	                "$svnrepo/pr ject/branches/fun plugin" &&
 	svn cp -m "more fun!" "$svnrepo/pr ject/branches/fun plugin" \
 	                      "$svnrepo/pr ject/branches/more fun plugin!" &&
+	svn cp -m "scary" "$svnrepo/pr ject/branches/fun plugin" \
+	              "$svnrepo/pr ject/branches/$scary_uri" &&
 	start_httpd
 	'
 
@@ -23,6 +29,7 @@ test_expect_success 'test clone with funky branch names' '
 	cd project &&
 		git rev-parse "refs/remotes/fun%20plugin" &&
 		git rev-parse "refs/remotes/more%20fun%20plugin!" &&
+		git rev-parse "refs/remotes/$scary_ref" &&
 	cd ..
 	'
 
@@ -35,6 +42,15 @@ test_expect_success 'test dcommit to funky branch' "
 	cd ..
 	"
 
+test_expect_success 'test dcommit to scary branch' '
+	cd project &&
+	git reset --hard "refs/remotes/$scary_ref" &&
+	echo urls are scary >> foo &&
+	git commit -m "eep" -- foo &&
+	git svn dcommit &&
+	cd ..
+	'
+
 stop_httpd
 
 test_done

Original file line number	Diff line number	Diff line change
`@@ -4025,6 +4025,7 @@ sub gs_do_switch {`
`4025`	`4025`	`}`
`4026`	`4026`	`}`
`4027`	`4027`	`$ra \|\|= $self;`
	`4028`	`+ $url_b = escape_url($url_b);`
`4028`	`4029`	`my $reporter = $ra->do_switch($rev_b, '', 1, $url_b, $editor, $pool);`
`4029`	`4030`	`my @lock = $SVN::Core::VERSION ge '1.2.0' ? (undef) : ();`
`4030`	`4031`	`$reporter->set_path('', $rev_a, 0, @lock, $pool);`