git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCHv2 4/5] Git.pm: add interface for git credential command

From
Michal Nazarewicz <mina86@mina86.com>
Date
Feb 8, 2013, 01:37 UTC
Message-ID
<xa1t6223poum.fsf@mina86.com>
In-Reply-To
<7v7gmj66fq.fsf@alter.siamese.dyndns.org>
On Fri, Feb 08 2013, Junio C Hamano wrote:
> I'd actually be more worried about the error checking issue
> Peff raised during his review.  I have a feeling that "when in doubt,
> do not cause harm" is a more prudent way to go than "I do not know,
> so I'll let anything pass".

I can implement whatever checking you wish, just tell me what to do in corner cases I've listed. ;)

-- 
Best regards,                                         _     _
.o. | Liege of Serenely Enlightened Majesty of      o' \,=./ `o
..o | Computer Science,  Michał “mina86” Nazarewicz    (o o)
ooo +----<email/xmpp: mpn@google.com>--------------ooO--(_)--Ooo--
Previous: Junio C HamanoNext: Jeff King
Message 8 of 11 in “[PATCHv2 0/5] Make git-send-email use git-credential”
  1. Michal NazarewiczFeb 7, 2013
  2. 1/5 Git.pm: allow command_close_bidi_pipe to be called as methodMichal Nazarewicz, Feb 7, 2013
  3. 2/5 Git.pm: fix example in command_close_bidi_pipe documentationMichal Nazarewicz, Feb 7, 2013
  4. 3/5 Git.pm: allow pipes to be closed prior to calling command_close_bidi_pipeMichal Nazarewicz, Feb 7, 2013
  5. 4/5 Git.pm: add interface for git credential commandMichal Nazarewicz, Feb 7, 2013
  6. Matthieu MoyFeb 7, 2013
  7. Junio C HamanoFeb 7, 2013
  8. Michal NazarewiczFeb 8, 2013
  9. Jeff KingFeb 8, 2013
  10. 5/5 git-send-email: use git credential to obtain passwordMichal Nazarewicz, Feb 7, 2013
  11. Junio C HamanoFeb 7, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.