On Fri, Sep 16, 2016 at 7:12 AM, Kevin Wern <kevin.m.wern@xxxxxxxxx> wrote:
> static struct daemon_service daemon_service[] = {
> { "upload-archive", "uploadarch", upload_archive, 0, 1 },
> { "upload-pack", "uploadpack", upload_pack, 1, 1 },
> { "receive-pack", "receivepack", receive_pack, 0, 1 },
> + { "prime-clone", "primeclone", prime_clone, 0, 1 },
> };
I guess this is why you chose to implement a new command in 01/11,
simpler to be called from http-backend?
> + // prime-clone does not need --stateless-rpc and
> + // --advertise-refs options. Maybe it will in the future, but
> + // until then it seems best to do this instead of adding
> + // "dummy" options.
Stick to /* .. */
> + if (strcmp(svc->name, "prime-clone") != 0) {
> + argv_array_pushl(&argv, "--stateless-rpc",
> + "--advertise-refs", NULL);
> + }
We also have an exception for select_getanyfile() below. I think it's
time we add a function callback in struct rpc_service to run each
service the way they want. Then prime-clone won't need an exception
(neither does select_anyfile, mostly)
--
Duy