Re: [PATCHv3] git-send-email: add ~/.authinfo parsing
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 30, 2013, 00:34 UTC
- Message-ID
- <7v1ud3o6ep.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <5d18d777d6ddf6f01bbf460f37af637d3dc28ed5.1359503987.git.mina86@mina86.com>
Michal Nazarewicz <mpn@google.com> writes:
Show 13 quoted lines
>> 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.