git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* git-svn doesn't like !! in the url
@ 2008-01-07  7:10 Michael J. Cohen
  2008-01-07 10:30 ` Eric Wong
  0 siblings, 1 reply; 5+ messages in thread
From: Michael J. Cohen @ 2008-01-07  7:10 UTC (permalink / raw)
  To: Git Mailing List

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: git-svn doesn't like !! in the url
  2008-01-07  7:10 git-svn doesn't like !! in the url Michael J. Cohen
@ 2008-01-07 10:30 ` Eric Wong
  2008-01-07 10:40   ` [PATCH] git-svn: clarify the "Ignoring error from SVN" piece Eric Wong
  2008-01-07 23:22   ` git-svn doesn't like !! in the url Junio C Hamano
  0 siblings, 2 replies; 5+ messages in thread
From: Eric Wong @ 2008-01-07 10:30 UTC (permalink / raw)
  To: Michael J. Cohen; +Cc: Git Mailing List, Junio C Hamano

"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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] git-svn: clarify the "Ignoring error from SVN" piece
  2008-01-07 10:30 ` Eric Wong
@ 2008-01-07 10:40   ` Eric Wong
  2008-01-08  0:28     ` Junio C Hamano
  2008-01-07 23:22   ` git-svn doesn't like !! in the url Junio C Hamano
  1 sibling, 1 reply; 5+ messages in thread
From: Eric Wong @ 2008-01-07 10:40 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Git Mailing List, Michael J. Cohen

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

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: git-svn doesn't like !! in the url
  2008-01-07 10:30 ` Eric Wong
  2008-01-07 10:40   ` [PATCH] git-svn: clarify the "Ignoring error from SVN" piece Eric Wong
@ 2008-01-07 23:22   ` Junio C Hamano
  1 sibling, 0 replies; 5+ messages in thread
From: Junio C Hamano @ 2008-01-07 23:22 UTC (permalink / raw)
  To: Eric Wong; +Cc: Michael J. Cohen, Git Mailing List

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'?

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] git-svn: clarify the "Ignoring error from SVN" piece
  2008-01-07 10:40   ` [PATCH] git-svn: clarify the "Ignoring error from SVN" piece Eric Wong
@ 2008-01-08  0:28     ` Junio C Hamano
  0 siblings, 0 replies; 5+ messages in thread
From: Junio C Hamano @ 2008-01-08  0:28 UTC (permalink / raw)
  To: Eric Wong; +Cc: Git Mailing List, Michael J. Cohen

Will apply to 'maint' ;-)

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2008-01-08  0:29 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-01-07  7:10 git-svn doesn't like !! in the url Michael J. Cohen
2008-01-07 10:30 ` Eric Wong
2008-01-07 10:40   ` [PATCH] git-svn: clarify the "Ignoring error from SVN" piece Eric Wong
2008-01-08  0:28     ` Junio C Hamano
2008-01-07 23:22   ` git-svn doesn't like !! in the url Junio C Hamano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).