Re: [PATCH 2/5] make credential helpers builtins
- From
Jeff King <peff@peff.net>
- Date
- Aug 13, 2020, 15:14 UTC
- Message-ID
- <20200813151410.GA892741@coredump.intra.peff.net>
- In-Reply-To
- <20200813150839.GB2244@syl.lan>
On Thu, Aug 13, 2020 at 11:08:39AM -0400, Taylor Blau wrote:
Show 14 quoted lines
> On Thu, Aug 13, 2020 at 10:58:55AM -0400, Jeff King wrote: > > There's no real reason for credential helpers to be separate binaries. I > > did them this way originally under the notion that helper don't _need_ > > to be part of Git, and so can be built totally separately (and indeed, > > the ones in contrib/credential are). But the ones in our main Makefile > > build on libgit.a, and the resulting binaries are reasonably large. > > Could you clarify which helpers you mean here? Git's own > credential-cache and store make sense to convert, but the helpers in > contrib definitely don't. > > For what it's worth, I'm almost positive that you mean the in-tree > helpers (where in-tree means "in git.git but not in contrib"), in which > case I'm in favor of this direcftion.
Yes, I mean the helpers in the Makefile that we build and install by default with a regular "make".
-Peff