# git-svn doesn't like !! in the url

5 messages from 2008-01-07 to 2008-01-08. Participants: Michael J. Cohen, Eric Wong, Junio C Hamano.
Thread: https://gitlist.dev/t/11504

## Michael J. Cohen, 2008-01-07 07:10

Subject: git-svn doesn't like !! in the url
Message-ID: <EA596F68-D87B-49AD-9DEF-2C2E07127BDE@gmail.com>
URL: https://gitlist.dev/e/EA596F68-D87B-49AD-9DEF-2C2E07127BDE%40gmail.com

```
I think it needs to urlencode the items before it passes it back to  
the server.

W: Ignoring error from SVN, path probably does not exist: (175007):  
HTTP Path Not Found: REPORT request failed on '/wowace/!svn/bc/100/ 
trunk/!!!LeaveMeAloneLibraries': '/wowace/!svn/bc/100/trunk/!!! 
LeaveMeAloneLibraries' path not found

works fine with svn co.

-mjc

```

## Eric Wong, 2008-01-07 10:30

Subject: Re: git-svn doesn't like !! in the url
Message-ID: <20080107103040.GA28557@soma>
URL: https://gitlist.dev/e/20080107103040.GA28557%40soma
In-Reply-To: <EA596F68-D87B-49AD-9DEF-2C2E07127BDE@gmail.com>

```
"Michael J. Cohen" <michael.joseph.cohen@gmail.com> wrote:
> I think it needs to urlencode the items before it passes it back to  
> the server.
> 
> W: Ignoring error from SVN, path probably does not exist: (175007):  
> HTTP Path Not Found: REPORT request failed on '/wowace/!svn/bc/100/ 
> trunk/!!!LeaveMeAloneLibraries': '/wowace/!svn/bc/100/trunk/!!! 
> LeaveMeAloneLibraries' path not found
> 
> works fine with svn co.

Hi,

That was just a warning message (hence "W:").  If the path you're
tracking did not have a revision between r0 - r100 in the repository;
then it'll spew that warning once (it used to spew it a lot more).

git-svn looks at old history in chunks starting at the beginning of the
repository to avoid pulling (potentially) hundreds of thousands of
revisions + filenames into memory all at once.

As long as git-svn started importing revisions when the project started,
everything should be alright.



I just dug up cfbe7ab333d68790eb37341e30f040f99cef6af7 and that
should've escaped everything that needs to be urlencoded for HTTP(S).
(you were also the one that noticed the need for that one, too :).

I also just noticed that changeset didn't make it into 1.5.3.7 nor
maint, however...

Junio: if there are plans for 1.5.3.8, could you please add
cfbe7ab333d68790eb37341e30f040f99cef6af7 to it?  Thanks.

-- 
Eric Wong

```

## Eric Wong, 2008-01-07 10:40

Subject: [PATCH] git-svn: clarify the "Ignoring error from SVN" piece
Message-ID: <20080107104040.GA1594@soma>
URL: https://gitlist.dev/e/20080107104040.GA1594%40soma
In-Reply-To: <20080107103040.GA28557@soma>

```
I've heard of several users puzzled by this, and it sometimes it
appears as if git-svn is doing nothing on slower connections and
larger repositories.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
---
 git-svn.perl |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/git-svn.perl b/git-svn.perl
index 2c97b05..3308fe1 100755
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -4085,6 +4085,10 @@ sub skip_unknown_revs {
 			warn "W: Ignoring error from SVN, path probably ",
 			     "does not exist: ($errno): ",
 			     $err->expanded_message,"\n";
+			warn "W: Do not be alarmed at the above message ",
+			     "git-svn is just searching aggressively for ",
+			     "old history.\n",
+			     "This may take a while on large repositories\n";
 			$ignored_err{$err_key} = 1;
 		}
 		return;
-- 
Eric Wong

```

## Junio C Hamano, 2008-01-07 23:22

Subject: Re: git-svn doesn't like !! in the url
Message-ID: <7vr6gtxlto.fsf@gitster.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vr6gtxlto.fsf%40gitster.siamese.dyndns.org
In-Reply-To: <20080107103040.GA28557@soma>

```
Eric Wong <normalperson@yhbt.net> writes:

> I just dug up cfbe7ab333d68790eb37341e30f040f99cef6af7 and that
> should've escaped everything that needs to be urlencoded for HTTP(S).
> (you were also the one that noticed the need for that one, too :).
>
> I also just noticed that changeset didn't make it into 1.5.3.7 nor
> maint, however...
>
> Junio: if there are plans for 1.5.3.8, could you please add
> cfbe7ab333d68790eb37341e30f040f99cef6af7 to it?  Thanks.

Surely I can cherry pick that commit.  Are you sure the test in
it (t9118) does not depend on any other new features that appear
on 'master'?

```

## Junio C Hamano, 2008-01-08 00:28

Subject: Re: [PATCH] git-svn: clarify the "Ignoring error from SVN" piece
Message-ID: <7vir25xirf.fsf@gitster.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vir25xirf.fsf%40gitster.siamese.dyndns.org
In-Reply-To: <20080107104040.GA1594@soma>

```
Will apply to 'maint' ;-)

```
