Re: [PATCH] gitweb/INSTALL: GITWEB_CONFIG_SYSTEM is for backward compatibility

2013-04-17 Thread Junio C Hamano
Drew Northup writes: >> That makes it sound like the "per instance overrides common overrides >> built-in" cascading is what is unusual and what we need to apologize >> for. > > I don't think were apologizing for anything. It is helpful to say "we > do some things differently here and don't plan

Re: [PATCH] gitweb/INSTALL: GITWEB_CONFIG_SYSTEM is for backward compatibility

2013-04-17 Thread Drew Northup
On Tue, Apr 16, 2013 at 6:26 PM, Jonathan Nieder wrote: > Drew Northup wrote: > >> This is unobtrusive yet to the point. > > I agree with the spirit. > > [...] >> --- a/Documentation/gitweb.conf.txt >> +++ b/Documentation/gitweb.conf.txt >> @@ -55,7 +55,8 @@ following o

Re: [PATCH] gitweb/INSTALL: GITWEB_CONFIG_SYSTEM is for backward compatibility

2013-04-16 Thread Jonathan Nieder
Drew Northup wrote: > This is unobtrusive yet to the point. I agree with the spirit. [...] > --- a/Documentation/gitweb.conf.txt > +++ b/Documentation/gitweb.conf.txt > @@ -55,7 +55,8 @@ following order: > then fallback system-wide configuration file (defaults to

Re: [PATCH] gitweb/INSTALL: GITWEB_CONFIG_SYSTEM is for backward compatibility

2013-04-16 Thread Drew Northup
Forgot the S-O-B... On Tue, Apr 16, 2013 at 8:47 AM, Drew Northup wrote: > On Tue, Apr 16, 2013 at 8:26 AM, Jakub Narębski wrote: > >> Drew: gitweb(1) or gitweb.conf(5) solution is more involved, so >> perhaps something like that? > > > That or: (or both I supposehopefully not too mangled by

Re: [PATCH] gitweb/INSTALL: GITWEB_CONFIG_SYSTEM is for backward compatibility

2013-04-16 Thread Drew Northup
On Tue, Apr 16, 2013 at 8:26 AM, Jakub Narębski wrote: > Drew: gitweb(1) or gitweb.conf(5) solution is more involved, so > perhaps something like that? That or: (or both I supposehopefully not too mangled by Google's mail gadget) -- >8 -- Subject: [PATCH] Documentation/gitweb.conf.txt: Mov

[PATCH] gitweb/INSTALL: GITWEB_CONFIG_SYSTEM is for backward compatibility

2013-04-16 Thread Jakub Narębski
Signed-off-by: Jakub Narebski --- This can be either squashed with previous patch to gitweb/INSTALL, kept as separate patch or discarded. Drew: gitweb(1) or gitweb.conf(5) solution is more involved, so perhaps something like that? gitweb/INSTALL |3 ++- 1 file changed, 2 insertions(+), 1 de