{"thread":{"id":"15331","subject":"Unable to clone svn repository","startedAt":"2008-09-02T14:58:15Z","lastAt":"2008-09-07T04:04:01Z","messageCount":2,"participants":["m.skoric@web.de","Eric Wong"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"89509","messageId":"1293837402@web.de","threadId":"15331","inReplyTo":null,"subject":"Unable to clone svn repository","fromName":"","fromEmail":"m.skoric@web.de","sentAt":"2008-09-02T14:58:15Z","receivedAt":"2008-09-02T14:58:15Z","isPatch":false,"sender":{"key":"m.skoric@web.de","avatar":null},"body":"Hi guys,\n\nim getting an error when im trying to clone a big svn repository. trunk and a couple of branches worked before\ngit crashed.\n\ngit Version 1.6.0.1\n\ncommand: git-svn clone https://SECRETURL --trunk=trunk --branches=branches --tags=tags\nerror:\n\nFound possible branch point: https://SECRETURL/branches/dirk/Abo-Uebernahme (Bug #994) => https://SECRETURL/branches/Abo-Uebernahme (Bug #994), 341\nInitializing parent: Abo-Uebernahme (Bug #994)@341\nW: Ignoring error from SVN, path probably does not exist: (175002): RA layer request failed: REPORT request failed on '/svn/tixx1/!svn/bc/101': REPORT of '/svn/tixx1/!svn/bc/101': Could not read chunk size: Secure connection truncated (https://SECRETURL)\nW: Do not be alarmed at the above message git-svn is just searching aggressively for old history.\nThis may take a while on large repositories\nFound possible branch point: https://SECRETURL/trunk => https://SECRETURL/branches/dirk/Abo-Uebernahme (Bug #994), 203\nFound branch parent: (Abo-Uebernahme (Bug #994)@341) bb831869748c98bf97d105c5894ae65331c95c08\nBad URL passed to RA layer: Malformed URL for repository at /usr/bin/git-svn line 3792\n\nMaybe a https problem -> \"Secure connection truncated (https://SECRETURL)\" ?\nAnyone has an idea and can help me?\n\nGreetz ms\n____________________________________________________________________\nPsssst! Schon vom neuen WEB.DE MultiMessenger gehört? \nDer kann`s mit allen: http://www.produkte.web.de/messenger/?did=3123\n"},{"id":"89941","messageId":"20080907040401.GA19659@untitled","threadId":"15331","inReplyTo":"1293837402@web.de","subject":"Re: Unable to clone svn repository","fromName":"Eric Wong","fromEmail":"normalperson@yhbt.net","sentAt":"2008-09-07T04:04:01Z","receivedAt":"2008-09-07T04:04:01Z","isPatch":false,"sender":{"key":"e@80x24.org","avatar":null},"body":"m.skoric@web.de wrote:\n> Hi guys,\n\nHi,\n\n> Found possible branch point: https://SECRETURL/trunk => https://SECRETURL/branches/dirk/Abo-Uebernahme (Bug #994), 203\n> Found branch parent: (Abo-Uebernahme (Bug #994)@341) bb831869748c98bf97d105c5894ae65331c95c08\n> Bad URL passed to RA layer: Malformed URL for repository at /usr/bin/git-svn line 3792\n\nThe following patch should fix it.  The strange URL with '#' and ')'\nthrew it off.\n\nI've also pushed this out to git://git.bogomips.org/git-svn.git\n\n>From f4392df485b197a0797cc503b1e1a71350abfe56 Mon Sep 17 00:00:00 2001\nFrom: Eric Wong <normalperson@yhbt.net>\nDate: Sat, 6 Sep 2008 20:18:18 -0700\nSubject: [PATCH] git-svn: fix handling of even funkier branch names\n\nApparently do_switch() tolerates the lack of escaping in less\nfunky branch names.  For the really strange and scary ones, we\nneed to escape them properly.  It strangely maintains compatible\nwith the existing handling of branch names with spaces and\nexclamation marks.\n\nReported-by: m.skoric@web.de ($gmane/94677)\nSigned-off-by: Eric Wong <normalperson@yhbt.net>\n---\n git-svn.perl                          |    1 +\n t/t9118-git-svn-funky-branch-names.sh |   16 ++++++++++++++++\n 2 files changed, 17 insertions(+), 0 deletions(-)\n\ndiff --git a/git-svn.perl b/git-svn.perl\nindex ee3f5ed..ecacf74 100755\n--- a/git-svn.perl\n+++ b/git-svn.perl\n@@ -4025,6 +4025,7 @@ sub gs_do_switch {\n \t\t}\n \t}\n \t$ra ||= $self;\n+\t$url_b = escape_url($url_b);\n \tmy $reporter = $ra->do_switch($rev_b, '', 1, $url_b, $editor, $pool);\n \tmy @lock = $SVN::Core::VERSION ge '1.2.0' ? (undef) : ();\n \t$reporter->set_path('', $rev_a, 0, @lock, $pool);\ndiff --git a/t/t9118-git-svn-funky-branch-names.sh b/t/t9118-git-svn-funky-branch-names.sh\nindex 3281cbd..43ceb75 100755\n--- a/t/t9118-git-svn-funky-branch-names.sh\n+++ b/t/t9118-git-svn-funky-branch-names.sh\n@@ -6,6 +6,10 @@\n test_description='git-svn funky branch names'\n . ./lib-git-svn.sh\n \n+# Abo-Uebernahme (Bug #994)\n+scary_uri='Abo-Uebernahme%20%28Bug%20%23994%29'\n+scary_ref='Abo-Uebernahme%20(Bug%20#994)'\n+\n test_expect_success 'setup svnrepo' '\n \tmkdir project project/trunk project/branches project/tags &&\n \techo foo > project/trunk/foo &&\n@@ -15,6 +19,8 @@ test_expect_success 'setup svnrepo' '\n \t                \"$svnrepo/pr ject/branches/fun plugin\" &&\n \tsvn cp -m \"more fun!\" \"$svnrepo/pr ject/branches/fun plugin\" \\\n \t                      \"$svnrepo/pr ject/branches/more fun plugin!\" &&\n+\tsvn cp -m \"scary\" \"$svnrepo/pr ject/branches/fun plugin\" \\\n+\t              \"$svnrepo/pr ject/branches/$scary_uri\" &&\n \tstart_httpd\n \t'\n \n@@ -23,6 +29,7 @@ test_expect_success 'test clone with funky branch names' '\n \tcd project &&\n \t\tgit rev-parse \"refs/remotes/fun%20plugin\" &&\n \t\tgit rev-parse \"refs/remotes/more%20fun%20plugin!\" &&\n+\t\tgit rev-parse \"refs/remotes/$scary_ref\" &&\n \tcd ..\n \t'\n \n@@ -35,6 +42,15 @@ test_expect_success 'test dcommit to funky branch' \"\n \tcd ..\n \t\"\n \n+test_expect_success 'test dcommit to scary branch' '\n+\tcd project &&\n+\tgit reset --hard \"refs/remotes/$scary_ref\" &&\n+\techo urls are scary >> foo &&\n+\tgit commit -m \"eep\" -- foo &&\n+\tgit svn dcommit &&\n+\tcd ..\n+\t'\n+\n stop_httpd\n \n test_done\n-- \nEric Wong\n"}]}