From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 30 Jan 2013 00:34:22 GMT
Subject: Re: [PATCHv3] git-send-email: add ~/.authinfo parsing
Message-ID: <7v1ud3o6ep.fsf@alter.siamese.dyndns.org>
In-Reply-To: <5d18d777d6ddf6f01bbf460f37af637d3dc28ed5.1359503987.git.mina86@mina86.com>

Michal Nazarewicz <mpn@google.com> writes:

>> Is it bad to use Net::Netrc instead?  This looks like exactly the
>> use case that module was written for, no?
>
> I don't think that's the case.  For one, Net::Netrc does not seem to
> process port number.
>
> There is a Text::Authinfo module but it just uses Text::CSV.
>
> I can change the code to use Net::Netrc, but I dunno if that's really
> the best option, since I feel people would expect parsing to be
> somehow compatible with
> <http://www.gnu.org/software/emacs/manual/html_node/gnus/NNTP.html>
> rather than the original .netrc file format.

Thanks for pushing back (I wish more contributors did so when I
suggest nonsense ;-)); you are right that both canned modules are
lacking.

Will queue.  Thanks.