All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Wong <normalperson@yhbt.net>
To: Junio C Hamano <gitster@pobox.com>
Cc: Boaz Stuller <boazstuller@gmail.com>, git@vger.kernel.org
Subject: Re: Problem with git-svn
Date: Wed, 20 Aug 2008 23:34:48 -0700	[thread overview]
Message-ID: <20080821063448.GA2392@untitled> (raw)
In-Reply-To: <5979e28c0808201045i2c525d7dx7fc8d70a67e9c9ef@mail.gmail.com>

Boaz Stuller <boazstuller@gmail.com> wrote:
> On Wed, Aug 20, 2008 at 4:11 AM, Eric Wong <normalperson@yhbt.net> wrote:
> > Boaz Stuller <boazstuller@gmail.com> wrote:
> >> I'm having a problem with git-svn.  I was connecting to a remote svn
> >> repository via the svn+ssh:// protocol using an embedded username in
> >> the url, i.e svn+ssh://boazstuller@svn.example.com/some/complicated/path.
> >>  When I upgraded to 1.6.0, 'git svn dcommit' stopped working and
> >> instead kept asking me for a password.   I tracked the problem down to
> >> the following commit:
> >>
> >> commit ba24e7457aa1f958370bbb67dfb97e3ec806fd4a
> >> Author: Eric Wong <normalperson@yhbt.net>
> >> Date:   Thu Aug 7 02:06:16 2008 -0700
> >>     git-svn: add ability to specify --commit-url for dcommit
> >>
> >> I don't know perl, but the problem seems to be where around line 446,
> >> '$gs->full_url' gets changed to  '$url'.  Apparently, $gs->full_url
> >> contained the embedded username but $url has it stripped out, i.e
> >> svn+ssh://svn.example.com/some/complicated/path , so ssh can't tell
> >> what username I'm trying to log in as.
> >
> > Hi Boaz, thanks for tracking this down.  The following patch should
> > fix your issue (and another potential one I noticed).
> >
> > I've lightly tested it and it appears to work for me.
> 
> I only tested with a couple commits too, but your patch works
> perfectly for me.

Junio: please apply my patch to maint, thank you.

-- 
Eric Wong

  reply	other threads:[~2008-08-21  6:35 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-19 13:41 Problem with git-svn Boaz Stuller
2008-08-20  8:11 ` Eric Wong
2008-08-20 17:45   ` Boaz Stuller
2008-08-21  6:34     ` Eric Wong [this message]
  -- strict thread matches above, loose matches on Subject: below --
2007-12-16 10:30 Pascal Obry
2007-12-16 13:56 ` Peter Baumann
2007-12-16 15:40   ` Pascal Obry
2007-12-19  8:27 ` Eric Wong
2007-12-19 11:27   ` Pascal Obry
2007-12-20 18:30     ` Eric Wong
2007-12-20 20:33       ` Pascal Obry
2007-12-21 15:42         ` Pascal Obry
2007-12-22  4:29         ` Eric Wong
2007-12-22 14:38           ` Pascal Obry
2007-12-20 20:34       ` Pascal Obry
2007-05-03 23:10 John Wiegley

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=20080821063448.GA2392@untitled \
    --to=normalperson@yhbt.net \
    --cc=boazstuller@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.