Re: [PATCH v2 1/3] git-svn: Support retrieving passwords with GIT_ASKPASS
- From
- Eric Wong <normalperson@yhbt.net>
- Date
- Feb 26, 2010, 10:05 UTC
- Message-ID
- <20100226100525.GC22579@dcvr.yhbt.net>
- In-Reply-To
- <1267142845-3040-1-git-send-email-lznuaa@gmail.com>
Frank Li <lznuaa@gmail.com> wrote:
Show 31 quoted lines
> git-svn reads passwords from an interactive terminal.
> This behavior cause GUIs to hang waiting for git-svn to
> complete
>
> Fix this problem by allowing a password-retrieving command
> to be specified in GIT_ASKPASS. SSH_ASKPASS is supported
> as a fallback when GIT_ASKPASS is not provided.
>
> Signed-off-by: Frank Li <lznuaa@gmail.com>
> ---
> git-svn.perl | 37 +++++++++++++++++++++++++++----------
> 1 files changed, 27 insertions(+), 10 deletions(-)
>
> diff --git a/git-svn.perl b/git-svn.perl
> index 265852f..cd39792 100755
> --- a/git-svn.perl
> +++ b/git-svn.perl
> @@ -31,6 +31,16 @@ if (! exists $ENV{SVN_SSH}) {
> }
> }
>
> +if (! exists $ENV{GIT_ASKPASS}) {
> + if (exists $ENV{SSH_ASKPASS}) {
> + $ENV{GIT_ASKPASS} = $ENV{SSH_ASKPASS};
> + if ($^O eq 'msys') {
> + $ENV{GIT_ASKPASS} =~ s/\\/\\\\/g;
> + $ENV{GIT_ASKPASS} =~ s/(.*)/"$1"/;
> + }
> + }
> +}
> +Hi Frank,
Since this logic isn't SVN-specific, can we get this in Git.pm and/or git-var so other tools can use it?
Thanks
-- Eric Wong