git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Eric Wong <normalperson@yhbt.net>
To: "Michael J. Cohen" <michael.joseph.cohen@gmail.com>
Cc: Git Mailing List <git@vger.kernel.org>,
	Junio C Hamano <gitster@pobox.com>
Subject: Re: git-svn doesn't like !! in the url
Date: Mon, 7 Jan 2008 02:30:40 -0800	[thread overview]
Message-ID: <20080107103040.GA28557@soma> (raw)
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

  reply	other threads:[~2008-01-07 10:31 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-07  7:10 git-svn doesn't like !! in the url Michael J. Cohen
2008-01-07 10:30 ` Eric Wong [this message]
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

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080107103040.GA28557@soma \
    --to=normalperson@yhbt.net \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=michael.joseph.cohen@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).