Re: git-daemon --inetd

On Thu, 15 Sep 2005, H. Peter Anvin wrote:
> 
> Could git-upload-pack be integrated into the git-daemon binary, so that 
> the exec isn't needed?

Hmm.. Looking a bit closer, that's not really very helpful. 

git-upload-pack ends up fork/exec'ing git-rev-list too, to generate the 
list of commits. And then it ends up finishing off with 
"git-pack-objects". 

Yes, we could make it all be one huge thing, and try to have lots of 
functional interfaces. But that would be really quite ugly and big changes 
(they'd have to be asynchronous and give incremental results etc etc).

Maybe a post-1.0 effort. For now, I think you need to have a minimal
execution environment inside the chroot. Not a lot: git-upload-pack,
git-rev-list and git-pack-objects.

Now, it turns out that git-rev-list really is the most complex of them 
all, largely due to --merge-order. If you define NO_OPENSSL, you'll get a 
much simpler execution environment.

Btw, I think --merge-order was cool, but its weaker cousin --topo-order is
what is actually _used_. Maybe we should deprecate --merge-order? Right
now the only real user is git-archimport, and I think that one too really
only wants topo-order too. For example, right now I think git-archimport 
won't actually work if you build without openssl.

Jon? Martin?

		Linus
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html