Re: [PATCH 1/2] Add a new option 'core.askpass'.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Knut Franke wrote:
> On Friday 27 August 2010 19:28:38 Junio C Hamano wrote:

>> Also this
>> does not feel like a "core." thing, but it may be just me.
>
> Where else could it go? It's not entirely http specific:
>
> $ grep git_getpass *.c
> connect.c:char *git_getpass(const char *prompt)
> http.c:                 user_pass = xstrdup(git_getpass("Password: "));
> http.c: ssl_cert_password = git_getpass("Certificate Password: ");
> http.c:                          * git_getpass is needed here because its very likely stdin/stdout are
> http.c:                          * but that is non-portable.  Using git_getpass() can at least be stubbed
> http.c:                         user_name = xstrdup(git_getpass("Username: "));
> imap-send.c:                    arg = git_getpass(prompt);
>
> Nor does it relate to a single command.

Perhaps "push.askpass" and "imap.askpass", maybe with the latter
defaulting to the former or something?  This would help preserve
the illusion that the git commands are sort of independent tools.

Haven't thought deeply about it at all, though.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]