On Thu, Jul 01, 2021 at 04:00:01PM +0200, Ævar Arnfjörð Bjarmason wrote:
> > * ab/serve-cleanup (2021-06-28) 8 commits
> > - upload-pack.c: convert to new serve.c "startup" config cb
> > - serve: add support for a "startup" git_config() callback
> > - serve.c: add trace2 regions for advertise & command
> > - serve.c: add call_{advertise,command}() indirection
> > - serve: use designated initializers
> > - transport: use designated initializers
> > - transport: rename "fetch" in transport_vtable to "fetch_refs"
> > - serve: mark has_capability() as static
> >
> > Code clean-up around "git serve".
>
> Jeff, what do you think about this version & the upload-pack.c changes?
I left some comments on the config-callback patches. I think it's an OK
direction.
I didn't read the final patch carefully. And I do think it needs to be
gone over very carefully to make sure there are not any subtleties,
given the problems we've had with config-reading there in the past. I'm
not enthused about doing so for what I consider to be a pretty
unexciting outcome. I.e., it feels a bit like high-risk churn for low
reward.
-Peff
