On Tue, Dec 15, 2020 at 06:07:56PM -0800, Jonathan Tan wrote: > +static int ls_refs_config(const char *var, const char *value, void *cb_data) > +{ > + struct ls_refs_data *data = cb_data; > + > + if (!strcmp("lsrefs.unborn", var)) > + data->allow_unborn = !strcmp(value, "allow") || > + !strcmp(value, "advertise"); What's the reason we would want this to be configurable? I would think we would just want it always on for the server, and then clients can choose to make us of it or not (and probably not by omitting the capability; the question is what they want to do with the information about HEAD, but that is true whether it is unborn or not, and is controlled by options like "clone -b"). -Peff