Re: [PATCH] gitweb: fill in gitweb configuration by Makefile

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



--- Martin Waitz <tali@xxxxxxxxxxxxxx> wrote:
> > This way, I can just copy gitweb.cgi into my web-server directory
> > and voila it works, since the variable definition file would
> > always be there.  This way:
> >   - no need to run make to build "gitweb.cgi" or "gitweb.pl" whatever
> >     you call it,
> 
> ok, but you have to call make anyway to build the rest of GIT,
> so this shouldn't be a big problem.
> 
> >   - no need to pollute your tree with site defined variables,
> 
> this is achieved by my patch, too.
> 
> >   - simple copy (cp) would install a working version, instead of
> >     the current cp + patch with local settings method.
> 
> this is achieved by my patch, too.

That's true, but the difference is that one needs to keep
a per-site-installation tree which keeps the configuration of
gitweb for that site (installation).

Since you're working on this, can you come up with an alternative way?
At this point I don't care either way, but _if_ there is an alternative
way, I'd rather we considered it.  (Possibly using a symbol table file
("package") to only keep those variables?)

The bash-sed editing is kind of, you have to admit, hackish.

   Luben

-
: 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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]