On Fri, 2013-04-12 at 09:08 -0700, Junio C Hamano wrote: > Jeff King <peff@xxxxxxxx> writes: > > >> How about "and make sure any Git configuration files", since there > >> might not be any Git configuration files. > > > > Yeah, that is better. Thanks. > > OK, then... > > -- >8 -- > Subject: [PATCH] doc: clarify that "git daemon --user=<user>" option does not export HOME=~user > > Signed-off-by: Jeff King <peff@xxxxxxxx> > Helped-by: W. Trevor King <wking@xxxxxxxxxx> > Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> > --- > Documentation/git-daemon.txt | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/git-daemon.txt b/Documentation/git-daemon.txt > index 7e5098a..2ac07ba 100644 > --- a/Documentation/git-daemon.txt > +++ b/Documentation/git-daemon.txt > @@ -147,6 +147,13 @@ OPTIONS > Giving these options is an error when used with `--inetd`; use > the facility of inet daemon to achieve the same before spawning > 'git daemon' if needed. > ++ > +Like many programs that switch user id, the daemon does not reset > +environment variables such as `$HOME` when it runs git programs, > +e.g. `upload-pack` and `receive-pack`. When using this option, you > +may also want to set and export `HOME` to point at the home > +directory of `<user>` before starting the daemon, and make sure any > +Git configuration files in that directory are readable by `<user>`. The "you may want to.." is perhaps a little understated given it will fail -EGOAWAY if git-daemon is started via init scripts if you don't. (but otoh, that's enough of a hint to anyone setting the thing up, no need to write paragraphs of legal-beagle boiler-plate for dinky bug;) -Mike -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html