From: Luben Tuikov <ltuikov@yahoo.com>
To: Jon Loeliger <jdl@freescale.com>, Junio C Hamano <junkio@cox.net>
Cc: Git List <git@vger.kernel.org>,
Martin Waitz <tali@admingilde.org>,
Matthias Lederhofer <matled@gmx.net>
Subject: Re: [PATCH] gitweb.cgi: Customization
Date: Wed, 2 Aug 2006 13:30:03 -0700 (PDT) [thread overview]
Message-ID: <20060802203003.61929.qmail@web31802.mail.mud.yahoo.com> (raw)
In-Reply-To: <1154535801.19994.15.camel@cashmere.sps.mot.com>
--- Jon Loeliger <jdl@freescale.com> wrote:
> On Tue, 2006-08-01 at 18:54, Junio C Hamano wrote:
>
> > In any case, I think tweaking gitweb.cgi from Makefile like
> > Martin Waitz did is as easy and clean for people who want to
> > customize; it should just be the matter of defining the
> > necessary params in config.mak.
>
> I disagree. I run multiple virtual web servers on one
> physical machine. Several of them run different gitweb
> instances, each with different configurations.
>
> With this "params in config.mk" approach, I have to
> run it multiple times, once for each web server I run.
That's exactly my sentiments.
> I _really_ would prefer an "include from ." feature
> where I can place the specific gitweb_config.pm parts
> in the same directory where gitweb.{pl,cgi} is installed.
Yes, this approach is simple and straightforward enough.
When upgrading, one only needs to do a "cp".
> We really need to separate out these config values
> from the gitweb.{pl,cgi} script itself. We _need_ to
> be able to update the gitweb script independently,
> and easily.
That's exactly where I was going with this patch.
I don't want to have to:
- configure httpd, or
- configure the environment of user "apache", or
- have a separate forked off branch in order to "configure"
gitweb at compile time for each gitweb instance I'm running,
- etc, etc, etc.
Configuration, especially of user space programs should be
completely dynamic. Compilation for user space programs should
be a separate process from configuration.
> > I do not think there is much difference between any of the
> > customization proposed so far (yours, Martin's and the one from
> > Matthias Lederhofer) from functionality and ease-of-use point of
> > view. They all try to make customization can be done in one
> > place, and the difference is mostly of taste, so I'd just pick
> > one from Martin.
>
> Let's just make sure it is a separate config file, please.
I thing your and mine deployment are pretty similar, thus warranting
a more flexible approach, withouth having to recompile or reconfigure
completely unrelated programs and users.
Luben
next prev parent reply other threads:[~2006-08-02 20:30 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-08-01 21:19 [PATCH] gitweb.cgi: Customization Luben Tuikov
2006-08-01 22:15 ` Junio C Hamano
2006-08-01 22:53 ` Luben Tuikov
2006-08-01 23:54 ` Junio C Hamano
2006-08-02 2:01 ` Luben Tuikov
2006-08-02 2:13 ` Martin Langhoff
2006-08-02 7:45 ` Junio C Hamano
2006-08-02 7:59 ` Martin Langhoff
2006-08-02 15:53 ` Matthias Lederhofer
2006-08-03 6:58 ` Martin Waitz
2006-08-03 16:24 ` Jeff King
2006-08-02 7:09 ` Matthias Lederhofer
2006-08-02 8:54 ` Matthias Lederhofer
2006-08-02 9:31 ` Martin Langhoff
2006-08-02 12:55 ` Matthias Lederhofer
2006-08-02 16:23 ` Jon Loeliger
2006-08-02 16:53 ` Jeff King
2006-08-02 17:13 ` Matthias Lederhofer
2006-08-02 17:34 ` Jakub Narebski
2006-08-02 17:43 ` Junio C Hamano
2006-08-02 20:30 ` Luben Tuikov [this message]
2006-08-02 19:40 ` Matthias Lederhofer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20060802203003.61929.qmail@web31802.mail.mud.yahoo.com \
--to=ltuikov@yahoo.com \
--cc=git@vger.kernel.org \
--cc=jdl@freescale.com \
--cc=junkio@cox.net \
--cc=matled@gmx.net \
--cc=tali@admingilde.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.