threads / discuss / 34721

netrc credential helper promotion out of contrib?

Subject: netrc credential helper promotion out of contrib?

## tl;dr

3 messages between Aug 18, 2013 and Sep 3, 2013.

replies: 2people: 2as markdown or json

Ted Zlatanov· Aug 18, 2013, 10:35 UTC · lore

A while has passed since contrib/credential/netrc was added. Is it OK to promote it to be part of the main installation? In that directory there's also gnome-keyring, osxkeychain, and wincred; I don't know if those are ready for promotion.

Thanks Ted

Junio C Hamano· Aug 20, 2013, 19:28 UTC · re: Ted Zlatanov · lore

Re: netrc credential helper promotion out of contrib?

Ted Zlatanov <tzz@lifelogs.com> writes:
> A while has passed since contrib/credential/netrc was added. Is it OK to
> promote it to be part of the main installation?

I gave it a quick glance, and it seems to be cleanly written, except that "EOHIPPUS" (End-of-Hippus? Eohippus the extinct horse?) looked a bit too strange to my taste ;-). It does not seem to use features older versions of Perl some people are stuck with do not support.

I do not mind seeing a patch that moves contrib/credential/netrc to credential/netrc and adjusts the top-level Makefile. The test script needs to be updated to fit the rest of t/ hierarchy better, though.

Ted Zlatanov· Sep 3, 2013, 15:12 UTC · re: Junio C Hamano · lore

Re: netrc credential helper promotion out of contrib?

On Tue, 20 Aug 2013 12:28:39 -0700 Junio C Hamano <gitster@pobox.com> wrote: 
JCH> Ted Zlatanov <tzz@lifelogs.com> writes:
>> A while has passed since contrib/credential/netrc was added. Is it OK to
>> promote it to be part of the main installation?

JCH> I gave it a quick glance, and it seems to be cleanly written, except JCH> that "EOHIPPUS" (End-of-Hippus? Eohippus the extinct horse?) looked JCH> a bit too strange to my taste ;-). It does not seem to use features JCH> older versions of Perl some people are stuck with do not support.

Thanks, I was careful about that. `EOF' as the end-of-document marker is a bit boring, don't you think? Well, I can use it here if you insist, but it seems OK to me.

JCH> I do not mind seeing a patch that moves contrib/credential/netrc to JCH> credential/netrc and adjusts the top-level Makefile. The test JCH> script needs to be updated to fit the rest of t/ hierarchy better, JCH> though.

OK, I will work on that patch.  Thanks for the reply.
Ted

← back to recent threads